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 Sync-Request version in package.json #662

Closed
wants to merge 1 commit into from

Conversation

mindlessroman
Copy link
Member

I created a similar PR on azure-pipelines-tasks but that may not be the exact right place. microsoft/azure-pipelines-tasks#13556

Issue #11136 on microsoft/azure-pipelines-tasks points out that sync-request is way more up-to-date than 3.0.1 - Is there any reason not to advance the version for this library? At a minimum changing it to ^3.0.1, if not something higher?

I personally am experiencing similar issues with this erroring out when being run on a pipeline agent even if my version of package.json has a higher version.

Issue #11136 on microsoft/azure-pipelines-tasks points out that sync-request is way more up-to-date than 3.0.1 - Is there any reason not to advance the version for this library? At a minimum changing it to `^3.0.1`, if not something higher?

I personally am experiencing similar issues with this erroring out when being run on a pipeline agent even if my version of package.json has a higher version.
@alex-peck
Copy link
Contributor

3.0.1 is the last version that is supported for Node 6, which most tasks currently rely on. There is work in-flight to bring tasks up to Node 14.

I have bumped sync-request up to 6.1.0 as a part of this work here: #665

Also worth noting that 3.0.1 is the last version in the semver major 3 so ^3.0.1 wouldn't actually result in upgrading this package.

@mindlessroman
Copy link
Member Author

Fair enough! I'll close out this PR and watch out for yours. Thank you for taking a look @alex-peck.

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.

2 participants