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

Update Block Breaking Stage Pipeline #96

Conversation

ChampionAsh5357
Copy link
Contributor

@ChampionAsh5357 ChampionAsh5357 commented Jun 2, 2024

Updates the breaking stage pipeline to match what is currently within 1.20.6. Also add a bit more information about the events fired.


Preview URL: https://pr-96.neoforged-docs-previews.pages.dev

@ChampionAsh5357 ChampionAsh5357 requested a review from a team June 2, 2024 12:28
@neoforged-pages-deployments neoforged-pages-deployments bot deployed to neoforged-docs-previews (Preview) June 2, 2024 12:30 Active
@neoforged-pages-deployments
Copy link

Deploying with Cloudflare Pages

Name Result
Last commit: 0245d69ffb55756d5c91d34917d29c90667a38bf
Status: ✅ Deploy successful!
Preview URL: https://9378f355.neoforged-docs-previews.pages.dev
PR Preview URL: https://pr-96.neoforged-docs-previews.pages.dev

@Shadows-of-Fire
Copy link
Contributor

There will be incoming changes to this area when I rebase the BlockDropsEvent PR against 1.20.6; not sure if you want to wait for that or adjust it after the fact

@ChampionAsh5357
Copy link
Contributor Author

There will be incoming changes to this area when I rebase the BlockDropsEvent PR against 1.20.6; not sure if you want to wait for that or adjust it after the fact

I don't think that will change this pipeline in terms of how things are executed. It will only add an additional event line. So, unless the event is coming out within the next few days, I think just judge it on the current structure of NeoForge.

@IchHabeHunger54 IchHabeHunger54 added the update Previously correct, but now outdated information. label Jun 12, 2024
@ChampionAsh5357
Copy link
Contributor Author

Bundled in #108

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update Previously correct, but now outdated information.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants