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

Update axios and prepare for a 0.1.0 release #5

Merged
merged 7 commits into from
Jan 23, 2024
Merged

Update axios and prepare for a 0.1.0 release #5

merged 7 commits into from
Jan 23, 2024

Conversation

scsmith
Copy link
Contributor

@scsmith scsmith commented Jan 23, 2024

No description provided.

This also add with some field documentation.
This also uses a newer version of node so the NPM package version
becomes 2.
This follows the move to a newer version of node for creating the
project.
@scsmith
Copy link
Contributor Author

scsmith commented Jan 23, 2024

@sakgoyal would you take a look at this and see if it satisfies your needs? I don't think we really want to update typescript or make any changes to the automatically generates files at this point. However, we'll review those options once this is live.

@sakgoyal
Copy link

Axios needs to be as updated as possible. The older versions have security vulnerabilities.

I don't really care about the other dependencies as it doesn't really affect anything else.

@scsmith
Copy link
Contributor Author

scsmith commented Jan 23, 2024

Axios needs to be as updated as possible. The older versions have security vulnerabilities.

This should allow any version of axios < 2.0 whilst not forcing anyone to use a specific version of 1.x. We'll try and get this published today.

@scsmith scsmith merged commit 29ec743 into main Jan 23, 2024
6 checks passed
@scsmith scsmith deleted the axios-1 branch January 23, 2024 17:33
This was referenced Jan 23, 2024
@sakgoyal
Copy link

Oh I missed the "^" in the package version. Perfect. Thank you

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