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

generators: js: fix sinon dependency to 15.2.0, add node tests #875

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

Williangalvani
Copy link
Contributor

@Williangalvani Williangalvani commented Nov 20, 2023

this fixes a node12 issue with optional chaining in Sinon lib. check recent mavlink actions

I'm adding the js tests here because it is weird that it only gets to fail on the other repo.

@Williangalvani Williangalvani changed the title generators: js: fix sinon dependency to 17.0.0 generators: js: fix sinon dependency to 17.0.0, add node tests Nov 20, 2023
@Williangalvani Williangalvani force-pushed the fix-sinon branch 27 times, most recently from 12df171 to 862078a Compare November 20, 2023 15:38
@Williangalvani Williangalvani changed the title generators: js: fix sinon dependency to 17.0.0, add node tests generators: js: fix sinon dependency to 15.2.0, add node tests Nov 20, 2023
@peterbarker peterbarker merged commit 86a99fd into ArduPilot:master Nov 20, 2023
11 checks passed
@davidbuzz
Copy link
Contributor

lgtm

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.

5 participants