https://www.runatlantis.io/ logo
Join Slack
Powered by
This message was deleted.
# atlantis-contributors
s
This message was deleted.
👀 1
l
I noticed after I updated https://github.com/runatlantis/atlantis/pull/3595, and just assumed it was related to my change so I closed the PR
But it appears to happen on all PRs, the issue is in the initialization on netlify:
Copy code
11:29:26 PM: error Couldn't find any versions for vuepress-vite that matches 2.0.0-beta.50-pre.1
info Visit <https://yarnpkg.com/en/docs/cli/install> for documentation about this command.
11:29:26 PM: Error during Yarn install
11:29:26 PM: Build was terminated: dependency_installation script returned non-zero exit code: 1
11:29:26 PM: Failing build: Failed to install dependencies
To clarify, the CI on main itself do not appear to be failing, but rather on any new PRs (which it looks like runs a few more tests)
I think it may have been introduced by a renovate package update https://github.com/runatlantis/atlantis/pull/3593 ?
d
thank you for bringing it up, we might have to move to a stable version. It appears we are on a prerelease of vuepress
l
I'm not sure there is a stable version https://github.com/vuepress/vuepress-next; unless you go back to 1.X
p
we can’t really go back now, we did a bunch of changes related to v2
l
any way we can just change these all to 2.0.0-beta.66?
d
part of the problem is the website check is buried in the Go Tests and not triggering when it should be
l
I'm actually not sure why that wasn't done as part of https://github.com/runatlantis/atlantis/pull/3600. My understanding of https://github.com/vuepress/vuepress-next is that it's a monorepo of all vuepress2 stuff, which would include vuepress-vite
d
so I'm breaking it out into its own test today
✅ 1
looks like I can't change branch protections?
l
ah yeah that seems like what happened here
d
I'm cleaning it up
renovate made a mess
Okay everything is all green, waiting on the last docker build (which I removed)
auto merge is enabled, hopefully it goes through with approval
even without the change to the required check
nope 
Required statuses must pass before merging
@lkysow whenever you have time we need to remove 
website_link_check
as a required check, I moved it so it should be 
website / link_check
now
Open in Slack
PreviousNext
Powered by