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

Pin ArduinoFFT version #3800

Closed
wants to merge 1 commit into from

Conversation

willmmiles
Copy link
Member

The upstream repository has updated the develop branch to v2.0 which is not API compatible. Pin the last commit before the new version was merged.

The upstream repository has released an incompatible commit on that
branch.  Pin the last commit before the new version was merged.
@blazoncek
Copy link
Collaborator

blazoncek commented Mar 7, 2024

IMO we're better off accepting new API and modify AR usermod. @softhack007 your thoughts?

This was also my request there: kosme/arduinoFFT#74

@willmmiles
Copy link
Member Author

IMO we're better off accepting new API and modify AR usermod.

I agree that we should update to the new version soon. I suggested this patch as a safe, short term fix as the GitHub CI system is now reporting build failures for all PRs, and clean checkouts won't build anymore either unless your local PlatformIO has already cached the old version.

@blazoncek
Copy link
Collaborator

blazoncek commented Mar 7, 2024

Resolved by using release version 2.0.0 of ArduinoFFT.
Needed change was minor.

@blazoncek blazoncek closed this Mar 7, 2024
@softhack007
Copy link
Collaborator

Resolved by using release version 2.0.0 of ArduinoFFT. Needed change was minor.

Looks good 👍 wow it's still a surprise that ArduinoFFT got a new release - the first update after 3 years or so.

@willmmiles
Copy link
Member Author

Good enough for me - I didn't have hardware on hand to test any changes, I just wanted the CI to work :)

@willmmiles willmmiles deleted the pin-arduinofft-version branch March 16, 2024 17:41
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.

3 participants