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

Revert "More consistent return value for annotations" #53309

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Feb 13, 2024

Reverts #53281 as it appears to have broken CI on all subsequent commits: #53281 (comment)

According to the CI logs, it is this test that aborts early on in the rr-trace log:

Pkg.test(TEST_PKG.name; coverage = coverage_path)

@vtjnash vtjnash added ci Continuous integration rr trace included priority This should be addressed urgently merge me PR is reviewed. Merge when all tests are passing labels Feb 13, 2024
@tecosaur
Copy link
Contributor

Can we work out more about why/how #53281 has actually caused an issue? I'm struggling to work out how it could have caused the problem it seems to have.

@vtjnash vtjnash merged commit 81c3474 into master Feb 13, 2024
8 of 11 checks passed
@vtjnash vtjnash deleted the revert-53281-more-consistent-annotated-return-forms branch February 13, 2024 14:45
tecosaur added a commit to tecosaur/julia that referenced this pull request Feb 14, 2024
@giordano giordano removed the merge me PR is reviewed. Merge when all tests are passing label Feb 15, 2024
tecosaur added a commit to tecosaur/julia that referenced this pull request Feb 19, 2024
KristofferC pushed a commit to tecosaur/julia that referenced this pull request Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration priority This should be addressed urgently rr trace included
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants