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

feature/sc-106677/stop-ignoring-device-os-scripts-directory #2471

Merged

Conversation

busticated
Copy link
Contributor

@busticated busticated commented Jul 13, 2022

Problem

Our developer tools require a series of scripts to support p2 flashing and debugging (e.g. init_km4.gdb). Currently those files live within the ./scripts directory and are ignored when creating the toolchain dependency bundle

Solution

Stop ignoring the ./scripts directory when building the toolchain bundle

Steps to Test

see: https://github.com/particle-iot/cli/pull/44

References

https://app.shortcut.com/particle/story/106677/stop-ignoring-device-os-scripts-directory-when-building-toolchain-bundle
https://github.com/particle-iot/cli#development:~:text=How%20to%20exclude%20files%20and%20directories%20from%20toolchain%20dependency%20bundles


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@technobly technobly added this to the 5.0.0 milestone Jul 13, 2022
scripts directory now contains p2-related debugging scripts needed to drive Workbench UX
@busticated busticated force-pushed the feature/sc-106677/stop-ignoring-device-os-scripts-directory branch from 2b0ac27 to bffadf1 Compare July 13, 2022 17:00
@busticated busticated marked this pull request as ready for review July 13, 2022 17:00
@technobly technobly merged commit 4939976 into develop Jul 13, 2022
@technobly technobly deleted the feature/sc-106677/stop-ignoring-device-os-scripts-directory branch July 13, 2022 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants