-
-
Notifications
You must be signed in to change notification settings - Fork 544
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
Bump cspell to 8.3.2 #2537
Bump cspell to 8.3.2 #2537
Conversation
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
Pull Request Test Coverage Report for Build 7423435531
💛 - Coveralls |
Thanks for looking into this. I was mislead by seemingly passing pipelines in #2531 which, as far as I understand the lock-file, maps both https://github.com/fluentassertions/fluentassertions/actions/runs/7378193454/job/20073037546 We need a way to distinguish between failing due to spelling mistakes and failing due to anything else... |
Hmm.. I don't get why it keeps failing on develop? Locally it works (for me) |
It already started failing with #2528 Reading more of yarnpkg/yarn#4812 and isaacs/jackspeak#5 it sounds like a yarn bug, that has been fixed in newer versions of yarn. natemcmaster/Yarn.MSBuild#22 (comment)
|
Ok.. e.g. something like: Node($"{YarnCli} --silent install", workingDirectory: RootDirectory,
logger: (_, msg) =>
{
if (msg.Contains("warning Pattern")) Information(msg);
else Error(msg);
}); or even: Node($"{YarnCli} --silent install", workingDirectory: RootDirectory,
logger: (_, msg) =>
{
if (!msg.Contains("warning Pattern")) Error(msg);
}); |
Running
Normally that implies to me, that I should start looking for an alternative to Brainstorming:
|
What about ignoring for now and hope for nuke-build/nuke#1151 ? |
I'm fine with ignoring for now, I'm just thinking about the future. I'm also subscribing to that PR, but as I understand it, it doesn't include yarn, but expects it to be present on the system. |
Yes, we can switch to NPM as well, but we have to investiage whether we can do that without having to install NPM on the build agent. |
We probably can mimic what this script does: https://www.npmjs.com/package/npm/v/10.2.5#direct-download |
Reason:
Before it had a strange error when running
yarn install
(GHA build and locally)After googling this error I found this issue: yarnpkg/yarn#4812
And one of the tips was: yarnpkg/yarn#4812 (comment)
So I deleted the lock file and run
yarn install
again, but this time it installs the latest 8 (8.3.2 at this moment) and the error was gone afterwardsIMPORTANT
./build.sh --target spellcheck
or.\build.ps1 --target spellcheck
before pushing and check the good outcome