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

fix(website): vuepress-next breaking change #3611

Merged
merged 9 commits into from
Jul 25, 2023
Merged

fix(website): vuepress-next breaking change #3611

merged 9 commits into from
Jul 25, 2023

Conversation

GenPage
Copy link
Member

@GenPage GenPage commented Jul 17, 2023

what

Fixes a breaking change introduced by vuepress-next team.

This also moves the website-link-checker into its own workflow and tweaks renovate config.

why

tests

  • I have tested my changes by running the new workflow.

references

closes #3603
closes #3608

@GenPage GenPage requested a review from a team as a code owner July 17, 2023 22:34
jamengual
jamengual previously approved these changes Jul 17, 2023
@github-actions github-actions bot added dependencies PRs that update a dependency file website labels Jul 18, 2023
@GenPage
Copy link
Member Author

GenPage commented Jul 18, 2023

so this was wild all I did was run yarn locally and select 66:

atlantis on  fix/website-tests [?] via 🐹 v1.19.3 on 🐳 v22.06.0-beta.0-926
➜ yarn
yarn install v1.22.19
[1/4] 🔍  Resolving packages...
warning @vuepress/[email protected]: Invalid version. Do not use it
warning [email protected]: Invalid version. Do not use it
Couldnt find any versions for "vuepress-vite" that matches "2.0.0-beta.50-pre.1"
? Please choose a version of "vuepress-vite" from this list:
 2.0.0-beta.66
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
warning "@vuepress/plugin-docsearch > @docsearch/react > @algolia/[email protected]" has unmet peer dependency "@algolia/client-search@>= 4.9.1 < 6".
warning "@vuepress/plugin-docsearch > @docsearch/react > @algolia/autocomplete-core > @algolia/[email protected]" has unmet peer dependency "search-insights@>= 1 < 3".
warning "@vuepress/plugin-docsearch > @docsearch/react > @algolia/autocomplete-core > @algolia/[email protected]" has unmet peer dependency "@algolia/client-search@>= 4.9.1 < 6".
warning "vuepress > [email protected]" has unmet peer dependency "vue@^3.3.4".
warning "vuepress > [email protected]" has unmet peer dependency "@vuepress/[email protected]".
warning "vuepress > vuepress-vite > @vuepress/theme-default > [email protected]" has unmet peer dependency "webpack@^5.0.0".
[4/4] 🔨  Building fresh packages...
success Saved lockfile.
✨  Done in 11.18s.

@jamengual
Copy link
Contributor

that is weird.

@GenPage
Copy link
Member Author

GenPage commented Jul 18, 2023

Found the issue: vuepress/core#1195

@GenPage GenPage enabled auto-merge (squash) July 18, 2023 04:04
@GenPage GenPage changed the title fix: website js tests triggering properly fix(website): vuepress-next breaking change Jul 18, 2023
jamengual
jamengual previously approved these changes Jul 18, 2023
auto-merge was automatically disabled July 24, 2023 23:08

Base branch was modified

@GenPage GenPage merged commit 70e5619 into main Jul 25, 2023
13 of 14 checks passed
@GenPage GenPage deleted the fix/website-tests branch July 25, 2023 03:03
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
* fix: vuepress-next breaking change

See vuepress/core#1195.

Other fixes:

* fix: website js tests triggering properly
* chore: clean up test workflows
* fix: yarn packages
* fix: renovate config & medium rate limit
* fix: don't need to require docker build for JS code
* fix: muffet config
* fix: properly name website test

---------

Co-authored-by: PePe Amengual <[email protected]>
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
* fix: vuepress-next breaking change

See vuepress/core#1195.

Other fixes:

* fix: website js tests triggering properly
* chore: clean up test workflows
* fix: yarn packages
* fix: renovate config & medium rate limit
* fix: don't need to require docker build for JS code
* fix: muffet config
* fix: properly name website test

---------

Co-authored-by: PePe Amengual <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies PRs that update a dependency file github-actions website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lack of GH App permission fails pre-hook but runs plan/apply as usual
2 participants