Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix list of modules for alias boards #4507

Merged
merged 2 commits into from
Mar 29, 2021

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
Member

@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.

tools/build_board_info.py Outdated Show resolved Hide resolved
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
Member

@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