Skip to content

Minor rules and declaration cleanups. #331

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

Merged
merged 3 commits into from
Apr 27, 2023
Merged

Minor rules and declaration cleanups. #331

merged 3 commits into from
Apr 27, 2023

Conversation

emilio
Copy link
Member

@emilio emilio commented Apr 27, 2023

No description provided.

@emilio
Copy link
Member Author

emilio commented Apr 27, 2023

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 48b8f0f has been approved by emilio

@bors-servo
Copy link
Contributor

⌛ Testing commit 48b8f0f with merge 64045a7...

bors-servo added a commit that referenced this pull request Apr 27, 2023
Minor rules and declaration cleanups.

None
@bors-servo
Copy link
Contributor

💔 Test failed - checks-github

@emilio emilio force-pushed the r-and-d-cleanups branch from e706b09 to bd27360 Compare April 27, 2023 12:04
@emilio
Copy link
Member Author

emilio commented Apr 27, 2023

@tiaanl @SimonSapin @jdm any objection on bumping MSRV to 1.63 which fixes the bugs that we were working around in #254?

Seems already old-ish enough, and allows #256 to get through too. I'm going to rejigger some of this code for nesting and it'd be good not having to think about the weird lifetime inference bug.

@SimonSapin
Copy link
Member

No objection!

@emilio
Copy link
Member Author

emilio commented Apr 27, 2023

@bors-servo r+

Alright then :)

@bors-servo
Copy link
Contributor

📌 Commit bd27360 has been approved by emilio

@emilio
Copy link
Member Author

emilio commented Apr 27, 2023

Well actually, since there's a release pending, let me do that first. @bors-servo r-

@emilio
Copy link
Member Author

emilio commented Apr 27, 2023

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit bd27360 has been approved by emilio

@bors-servo
Copy link
Contributor

⌛ Testing commit bd27360 with merge 20be7cc...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-github
Approved by: emilio
Pushing 20be7cc to master...

@bors-servo bors-servo merged commit 20be7cc into master Apr 27, 2023
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