Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Neradoc
Copy link

@Neradoc Neradoc commented Mar 29, 2021

Alias boards (different name for identical build) did not have their modules listed, due to looking for the directory by the name of the alias (doesn't exist), instead of the original it copies from.

circuitplayground_express_4h, circuitplayground_express_digikey_pycon2019, edgebadge, pyportal_pynt, gemma_m0_pycon2018, pewpew13.

Thanks @lesamouraipourpre for signaling.

Note: that only impacts future releases (stable or not)

@Neradoc Neradoc force-pushed the fix-alias-boards-modules branch from 8366b5c to 032209e Compare March 29, 2021 09:10
@Neradoc Neradoc force-pushed the fix-alias-boards-modules branch from 032209e to f143e5d Compare March 29, 2021 10:25
Copy link

@jepler jepler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think there's a slight simplification that can be done to improve the code and potentially detect future mistakes.

will raise an error during build if things as not as expected
as suggested by @jepler
@tannewt tannewt requested a review from jepler March 29, 2021 21:39
Copy link

@jepler jepler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you ! I didn't test.

@jepler jepler merged commit 38382bf into adafruit:main Mar 29, 2021
@Neradoc Neradoc deleted the fix-alias-boards-modules branch April 2, 2021 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants