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

Readme: note that config changes don't apply to already compiled code #8175

Closed
wants to merge 2 commits into from
Closed

Readme: note that config changes don't apply to already compiled code #8175

wants to merge 2 commits into from

Conversation

Kage-Yami
Copy link

Please write a short comment explaining your change (or "none" for internal only changes)

changelog: added a note to the Readme that config changes don't apply to already compiled code

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @llogiq (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 26, 2021
@Kage-Yami
Copy link
Author

This is documenting what was (re-)discovered as part of #8145.

Copy link
Contributor

@llogiq llogiq left a comment

Choose a reason for hiding this comment

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

Just a small suggestion for improvement, otherwise this looks good to merge.

README.md Outdated Show resolved Hide resolved
@llogiq
Copy link
Contributor

llogiq commented Dec 27, 2021

Thank you! @bors r+ squash

@bors
Copy link
Contributor

bors commented Dec 27, 2021

📌 Commit 67f73a3 has been approved by llogiq

@bors
Copy link
Contributor

bors commented Dec 27, 2021

⌛ Testing commit 67f73a3 with merge a72469a...

bors added a commit that referenced this pull request Dec 27, 2021
Readme: note that config changes don't apply to already compiled code

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: added a note to the Readme that config changes don't apply to already compiled code
@bors
Copy link
Contributor

bors commented Dec 27, 2021

💥 Test timed out

@llogiq
Copy link
Contributor

llogiq commented Dec 27, 2021

It appears that the test runners have some intermittent problems. I'll ask infra.

@llogiq
Copy link
Contributor

llogiq commented Dec 27, 2021

@bors retry

@bors
Copy link
Contributor

bors commented Dec 27, 2021

⌛ Testing commit 67f73a3 with merge 6746054...

bors added a commit that referenced this pull request Dec 27, 2021
Readme: note that config changes don't apply to already compiled code

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: added a note to the Readme that config changes don't apply to already compiled code
@bors
Copy link
Contributor

bors commented Dec 27, 2021

💔 Test failed - checks-action_remark_test

@llogiq
Copy link
Contributor

llogiq commented Dec 27, 2021

The remark step itself was successful. It seems something's wrong with github's test infra.

@matthiaskrgr
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Dec 27, 2021

⌛ Testing commit 67f73a3 with merge 44719b5...

bors added a commit that referenced this pull request Dec 27, 2021
Readme: note that config changes don't apply to already compiled code

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: added a note to the Readme that config changes don't apply to already compiled code
@bors
Copy link
Contributor

bors commented Dec 27, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 44719b5 to master...

@llogiq
Copy link
Contributor

llogiq commented Dec 28, 2021

It seems that bors hasn't successfully pushed to master despite the tests being OK.

@bors retry

@bors
Copy link
Contributor

bors commented Dec 28, 2021

⌛ Testing commit 67f73a3 with merge 3c1b3ec...

bors added a commit that referenced this pull request Dec 28, 2021
Readme: note that config changes don't apply to already compiled code

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: added a note to the Readme that config changes don't apply to already compiled code
@bors
Copy link
Contributor

bors commented Dec 28, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 3c1b3ec to master...

@llogiq
Copy link
Contributor

llogiq commented Dec 28, 2021

Ok, this seems to be a github UI problem where the PR is not marked as merge despite being the commit being merged to master (I just pulled and checked).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants