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.

Sign up for GitHub

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

Jump to bottom

yarn tests #130

Merged
merged 73 commits into from
Oct 20, 2023
Merged

yarn tests #130

merged 73 commits into from
Oct 20, 2023

Conversation

cdmistman
Copy link
Contributor

@cdmistman cdmistman commented Oct 3, 2023 •
edited
Loading

there's a bug with reading @org/package from the lockfile, which is why tests are failing

blocked on #126

@cdmistman
add some ignored things
a7ddb18
@cdmistman
use go 1.20
561a9d4
@cdmistman
go mod tidy
7d84029
@cdmistman
fix deprecated stdlib APIs from minor version releases cause thats to…
b6eda31 
…tally cool and normal
@cdmistman
fix deprecated flake output
23fb31a
@cdmistman
here wigo
83f56db
@cdmistman
full nodejs which-language testing
08b849f
@cdmistman
more tests
566fe49
@cdmistman
another test, cleanup
0030043
@cdmistman
move some things around, fix some fails
582f469
@cdmistman
fix testremove logic
ca8d934
@cdmistman
test guessing
26f373c
@cdmistman
better test layout
4ec1e89
@cdmistman
better search testing
28c0a1f
@cdmistman
better info testing
b2517d5
@cdmistman
better add testing
4a6f3b7
@cdmistman
better remove testing
5ee3957
@cdmistman
better lock testing
66dfac9
@cdmistman
better install testing
d314465
@cdmistman
better list testing
475cebe
@cdmistman
better guess testing
086adac
@cdmistman
Merge branch 'main' of github.com:replit/upm into cad/test-suite
eafb537
@cdmistman
add package managers to devshell for testing
9d477be
@cdmistman
exclude test suite from upm src
ec9eb4c
@cdmistman
skip test suites that dont have tests
4cd86c2
@cdmistman
cleanup
2e6c041
@cdmistman
add test-suite to ci
9f5f131
@cdmistman
add nix to test-suite task
69ffccc
@cdmistman
share nix store, run make generated
1bdfdb4
@cdmistman
fix dependencies list i think
96f78aa
@cdmistman cdmistman marked this pull request as ready for review October 12, 2023 19:43
@cdmistman cdmistman requested a review from a team as a code owner October 12, 2023 19:43
@cdmistman cdmistman requested review from airportyh and removed request for a team October 12, 2023 19:43
@cdmistman
Copy link
Contributor Author

😱 failing ci after merge???

@cdmistman
Copy link
Contributor Author

ah crap bad merge

airportyh
airportyh reviewed Oct 17, 2023
View reviewed changes
Copy link
Contributor

@airportyh airportyh left a comment

Choose a reason for hiding this comment

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

I am getting some test failures:

  • TestAdd/nodejs-yarn/many-deps/unlocked/all
  • TestList/nodejs-yarn/many-deps
  • TestLock/nodejs-yarn

Among them.

airportyh
airportyh approved these changes Oct 17, 2023
View reviewed changes
Copy link
Contributor

@airportyh airportyh left a comment

Choose a reason for hiding this comment

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

Sorry nm. I didn't have the nix-shell properly setup. Now it works.

@cdmistman cdmistman merged commit 9a5f808 into main Oct 20, 2023
1 check passed
@cdmistman cdmistman deleted the cad/yarn-tests branch October 20, 2023 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@airportyh airportyh airportyh approved these changes

Assignees
No one assigned
Labels
chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cdmistman @airportyh @blast-hardcheese