-
Notifications
You must be signed in to change notification settings - Fork 9.2k
Migrate to yarn to avoid npm workarounds #5503
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| name: build-infra-integration | ||
|
|
||
| on: | ||
| pull_request: {} | ||
| workflow_dispatch: {} | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v7 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - uses: actions/setup-node@v7 | ||
| with: | ||
| node-version: "24.x" | ||
|
|
||
| - name: Enable Corepack | ||
| run: corepack enable | ||
|
|
||
| - name: Install dependencies | ||
| run: yarn install --immutable | ||
|
|
||
| - name: Build latest published specification | ||
| if: ${{ hashFiles('versions/*.md') != '' }} | ||
| run: yarn build latest | ||
|
|
||
| - name: Build source specification | ||
| if: ${{ hashFiles('src/*.md') != '' }} | ||
| run: yarn build src | ||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -35,8 +35,11 @@ jobs: | |||||||
| with: | ||||||||
| node-version: "24.x" | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
|
||||||||
| - name: Enable Corepack | ||||||||
| run: corepack enable | ||||||||
|
|
||||||||
| - name: Install dependencies | ||||||||
| run: npm ci --allow-git=root | ||||||||
| run: yarn install --immutable | ||||||||
|
|
||||||||
| - uses: actions/checkout@v7 # checkout main branch of website repo | ||||||||
| with: | ||||||||
|
|
@@ -46,7 +49,7 @@ jobs: | |||||||
| path: deploy | ||||||||
|
|
||||||||
| - name: run main script | ||||||||
| run: npm run build | ||||||||
| run: yarn build | ||||||||
|
|
||||||||
| - name: Create Pull Request | ||||||||
| uses: peter-evans/create-pull-request@v8 | ||||||||
|
|
||||||||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -38,8 +38,11 @@ jobs: | |||||||
| with: | ||||||||
| node-version: "24.x" | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
|
||||||||
| - name: Enable Corepack | ||||||||
| run: corepack enable | ||||||||
|
|
||||||||
| - name: Install dependencies | ||||||||
| run: npm ci --allow-git=root | ||||||||
| run: yarn install --immutable | ||||||||
|
|
||||||||
| - uses: actions/checkout@v7 # checkout main branch of website repo | ||||||||
| with: | ||||||||
|
|
@@ -49,7 +52,7 @@ jobs: | |||||||
| path: deploy | ||||||||
|
|
||||||||
| - name: run main script | ||||||||
| run: npm run publish-schemas | ||||||||
| run: yarn publish-schemas | ||||||||
|
|
||||||||
| - name: Create Pull Request | ||||||||
| uses: peter-evans/create-pull-request@v8 | ||||||||
|
|
||||||||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -4,7 +4,7 @@ name: schema-test | |||||||
| # Issue: https://github.com/OAI/OpenAPI-Specification/pull/2489 | ||||||||
|
|
||||||||
| # | ||||||||
| # This workflow runs the npm test script to validate passing and failing | ||||||||
| # This workflow runs the Yarn test script to validate passing and failing | ||||||||
| # testcases for the metaschemas | ||||||||
| # | ||||||||
|
|
||||||||
|
|
@@ -27,10 +27,13 @@ jobs: | |||||||
| with: | ||||||||
| node-version: '24.x' | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
|
||||||||
| - name: Enable Corepack | ||||||||
| run: corepack enable | ||||||||
|
|
||||||||
| - name: Install dependencies | ||||||||
| run: npm ci --allow-git=root | ||||||||
| run: yarn install --immutable | ||||||||
|
|
||||||||
| - name: Run tests | ||||||||
| run: npm run test | ||||||||
| run: yarn test | ||||||||
| env: | ||||||||
| BASE: ${{ github.event.pull_request.base.ref }} | ||||||||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -21,8 +21,11 @@ jobs: | |||||||
| with: | ||||||||
| node-version: "24.x" | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
|
|
||||||||
| - name: Enable Corepack | ||||||||
| run: corepack enable | ||||||||
|
|
||||||||
| - name: Install dependencies | ||||||||
| run: npm ci --allow-git=root | ||||||||
| run: yarn install --immutable | ||||||||
|
|
||||||||
| - name: Validate Markdown | ||||||||
| run: npm run validate-markdown | ||||||||
| run: yarn validate-markdown | ||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,6 +5,7 @@ | |
| target | ||
| atlassian-ide-plugin.xml | ||
| node_modules/ | ||
| .yarn/ | ||
| deploy/ | ||
| deploy-preview/ | ||
| coverage/ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 24 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| nodeLinker: node-modules | ||
| approvedGitRepositories: | ||
| - https://github.com/OAI/build-infra.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.