Skip to content

Bootstrap code changes #1914

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 5 commits into from
Mar 3, 2025
Merged

Conversation

duckdoom5
Copy link
Contributor

@duckdoom5 duckdoom5 commented Feb 25, 2025

  • Added 'BootstrapPatch' folder to git ignore (just in case)
  • Abstracted some of the bootstrap code in preparation for adding auto generators for Decl/Type
  • More code cleanup

@duckdoom5
Copy link
Contributor Author

duckdoom5 commented Feb 25, 2025

Let me know if you'd prefer that I split the PR into smaller ones (like I did before).

Scratch that.. I'll do that anyway cus the build seems to be failing somehow

@duckdoom5 duckdoom5 force-pushed the fix/bootstrap-misc-changes branch from 0b0114c to 0a9a10a Compare February 25, 2025 14:34
@duckdoom5 duckdoom5 changed the title Misc changes Bootstrap code changes Feb 25, 2025
@duckdoom5 duckdoom5 force-pushed the fix/bootstrap-misc-changes branch 3 times, most recently from 7d56062 to 61355fa Compare February 25, 2025 16:13
@duckdoom5
Copy link
Contributor Author

This is also ready for review now FYI

@duckdoom5 duckdoom5 force-pushed the fix/bootstrap-misc-changes branch from 61355fa to d339bfe Compare February 27, 2025 07:48
duckdoom5 added a commit to duckdoom5/CppSharp that referenced this pull request Feb 27, 2025
@duckdoom5 duckdoom5 marked this pull request as draft February 27, 2025 09:15
@duckdoom5
Copy link
Contributor Author

Since this hasn't been merged yet, I'll just improve this commit a bit more today

@tritao
Copy link
Collaborator

tritao commented Feb 27, 2025

Since this hasn't been merged yet, I'll just improve this commit a bit more today

Cool, had just started a review actually 😄

@duckdoom5 duckdoom5 marked this pull request as ready for review March 3, 2025 08:47
@duckdoom5
Copy link
Contributor Author

@tritao This is ready for review now. Could still use some improvements, but that can be done in a later PR

@tritao tritao merged commit 7105da0 into mono:main Mar 3, 2025
10 checks passed
@duckdoom5 duckdoom5 deleted the fix/bootstrap-misc-changes branch March 3, 2025 17:25
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.

2 participants