Navigation Menu

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

Add tests for release script #11454

Merged

Conversation

sosukesuzuki
Copy link
Member

@sosukesuzuki sosukesuzuki commented Sep 5, 2021

Description

Adds tests for release script.

This PR adds test for one step (validate-new-version). In other PR, I'll add tests for other steps.

Checklist

Try the playground for this PR

@sosukesuzuki sosukesuzuki changed the title Add tests for release scripts Add tests for release script Sep 5, 2021
@sosukesuzuki sosukesuzuki marked this pull request as ready for review September 5, 2021 19:38
@sosukesuzuki
Copy link
Member Author

sosukesuzuki commented Sep 6, 2021

Why Dev_Package_Test failed? It failed on main too in my local machine...

@fisker
Copy link
Sponsor Member

fisker commented Sep 6, 2021

This means yarn.lock is outdate, and Prettier can't work with some new package.

@fisker
Copy link
Sponsor Member

fisker commented Sep 6, 2021

My guess, caused by https://github.com/acornjs/acorn/releases/tag/8.5.0, try regenerate lock file will see the problem.

@sosukesuzuki sosukesuzuki mentioned this pull request Sep 7, 2021
2 tasks
@sosukesuzuki
Copy link
Member Author

@fisker Thank you! I've created PR #11458

cache: "yarn"

- name: Install Dependencies
run: cd scripts/release && yarn install --frozen-lockfile
Copy link
Sponsor Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

48fc148 , works fine

@sosukesuzuki sosukesuzuki merged commit bf2a24f into prettier:main Sep 7, 2021
@sosukesuzuki sosukesuzuki deleted the add-tests-for-release-scripts branch September 7, 2021 05:53
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants