Skip to content

Small maintainance improvements in cargo and rustc#125

Merged
bors[bot] merged 3 commits intorust-marker:masterfrom
xFrednet:000-maintenance-magic
Apr 6, 2023
Merged

Small maintainance improvements in cargo and rustc#125
bors[bot] merged 3 commits intorust-marker:masterfrom
xFrednet:000-maintenance-magic

Conversation

@xFrednet
Copy link
Copy Markdown
Member

@xFrednet xFrednet commented Apr 4, 2023

This PR does some cleanup around the rustc driver and cargo. The commits should structure the changes pretty well.


I took a break after my exam week. It has been pretty good, but now I want and need to get back into marker. I hope that this PR is just the start of it :)


r? @Niki4tap Would you still have time to review my PRs? 🙃

@xFrednet xFrednet force-pushed the 000-maintenance-magic branch from 23d83b7 to a7e4ea1 Compare April 4, 2023 20:55
@xFrednet xFrednet added the S-waiting-on-review Status: Awaiting review label Apr 4, 2023
Copy link
Copy Markdown
Member

@Niki4tap Niki4tap left a comment

Choose a reason for hiding this comment

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

Looks great to me!

bors r+

side note: I'm a bit more busier than I've expected, but I should still be able to review your PRs. I think I have a local branch that covers some tasks from #81, but it needs a bit of a cleanup, I'll try to PR it as soon as I can.

@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Apr 6, 2023

Build succeeded:

@bors bors Bot merged commit 12228d1 into rust-marker:master Apr 6, 2023
@xFrednet xFrednet deleted the 000-maintenance-magic branch April 6, 2023 19:09
@xFrednet
Copy link
Copy Markdown
Member Author

xFrednet commented Apr 6, 2023

side note: I'm a bit more busier than I've expected, but I should still be able to review your PRs. I think I have a local branch that covers some tasks from #81, but it needs a bit of a cleanup, I'll try to PR it as soon as I can

No problem at all, thank you for the review! The same thing happened to me, my current courses are taking a bit more time than expected. But I'm motivated again, which is a good baseline I'd say :)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants