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

Testing #895 #896

Merged
merged 17 commits into from
Jul 29, 2024
Merged

Testing #895 #896

merged 17 commits into from
Jul 29, 2024

Conversation

CodyCBakerPhD
Copy link
Collaborator

Running #895 tests with CI permissions

@CodyCBakerPhD CodyCBakerPhD self-assigned this Jul 25, 2024
@CodyCBakerPhD
Copy link
Collaborator Author

npm ci is stalling out on all mac runners here despite nothing related being changed

Dailies yesterday ran fine, triggered on main just now to double check

Might be a live service npm issue

@CodyCBakerPhD
Copy link
Collaborator Author

Yeah even main is struggling ATM

I'll retry this again tomorrow

@CodyCBakerPhD
Copy link
Collaborator Author

It's been several days of failure now so seems to be a reliable issue

Trying to debug on this branch

@CodyCBakerPhD
Copy link
Collaborator Author

@rly See #901 for formal record, but just to clarify, local builds on my Mac work just fine on this branch, it's only the CI that is problematic as of 5 days ago

So I've disabled those in #900 (merged into this)

@CodyCBakerPhD CodyCBakerPhD marked this pull request as ready for review July 29, 2024 08:23
@CodyCBakerPhD CodyCBakerPhD requested a review from rly July 29, 2024 08:23
@rly
Copy link
Collaborator

rly commented Jul 29, 2024

Thanks for the formal record. That is useful. I will experiment with a couple things, but otherwise I think disabling the Mac builds until we have more time or help on how to dig deeper makes sense.

@CodyCBakerPhD CodyCBakerPhD merged commit 36bd34d into main Jul 29, 2024
11 checks passed
@CodyCBakerPhD CodyCBakerPhD deleted the staging branch July 29, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants