-
Notifications
You must be signed in to change notification settings - Fork 3k
Update maximum version of intelhex to 2.3.0 #14201
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
Update maximum version of intelhex to 2.3.0 #14201
Conversation
Can we remove the maximum version? I'd rather keep it "unpinned" so we don't have to update it again at some point in the future. |
I'm happy to do that! Should I also remove the minimum version or should I put 2.3.0? |
7d568d1
to
881fc18
Compare
@ladislas, thank you for your changes. |
881fc18
to
34c49de
Compare
34c49de
to
7983d81
Compare
This pull request has automatically been marked as stale because it has had no recent activity. @ARMmbed/mbed-os-maintainers, please start CI to get the PR merged. |
CI started |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Update minimum version of intelhex to 2.3.0.
Impact of changes
On macOS with Python 3.9.1, using intelhex in a script results in the following error:
The issue is well known, see:
python-intelhex/intelhex#45
With 2.3.0, the issue is fixed.
Migration actions required
none
Documentation
n/a
Pull request type
Test results
Reviewers
@0xc0170 @rwalton-arm @Patater (might be related to ARMmbed/mbed-tools#171)