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

Submit Racingway plugin. #5440

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Submit Racingway plugin. #5440

wants to merge 3 commits into from

Conversation

Abyeon
Copy link

@Abyeon Abyeon commented Feb 10, 2025

No description provided.

@bleatbot bleatbot enabled auto-merge (squash) February 10, 2025 04:36
@bleatbot
Copy link
Collaborator

bleatbot commented Feb 10, 2025

Outdated attempt

This is the first time that you have submitted a plugin here. Before the bot will build your plugin within the 'Build PR' check, someone from the approval team will need to enable builds for you.

Once this is enabled, the bot will automatically build the PR. Future iterations will not require an approval for building the PR, only merging.

Please hold!

Updated commit to be more up to date.
auto-merge was automatically disabled February 11, 2025 01:49

Head branch was pushed to by a user without write access

@bleatbot
Copy link
Collaborator

bleatbot commented Feb 11, 2025

Outdated attempt

This is the first time that you have submitted a plugin here. Before the bot will build your plugin within the 'Build PR' check, someone from the approval team will need to enable builds for you.

Once this is enabled, the bot will automatically build the PR. Future iterations will not require an approval for building the PR, only merging.

Please hold!

@philpax
Copy link
Contributor

philpax commented Feb 12, 2025

bleatbot, approve

@bleatbot
Copy link
Collaborator

bleatbot commented Feb 12, 2025

Outdated attempt

All builds OK!

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

Name Commit Status
✔️ Racingway [testing-live] 144b29f v0.0.1.0 - Repo (New plugin)
1 hidden need (known safe NuGet packages).
Show log - Review

@bleatbot bleatbot added new plugin This is a new plugin. size-large Diff for this PR is large. labels Feb 12, 2025
@philpax
Copy link
Contributor

philpax commented Feb 12, 2025

This plugin is now in the plugin review queue; we hope to get back to you soon!

@philpax philpax added completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. labels Feb 12, 2025
@Abyeon
Copy link
Author

Abyeon commented Feb 14, 2025

I've made some pretty significant changes. Should I wait for the review, or should I make a new branch and PR?

@philpax
Copy link
Contributor

philpax commented Feb 14, 2025

Feel free to update the PR, we'll review any updates :)

Reorganized a lot of code, removed lots of old debug stuff, overhauled the UI, and added a temporary race leaderboard of sorts.
@Abyeon
Copy link
Author

Abyeon commented Feb 14, 2025

Feel free to update the PR, we'll review any updates :)

Thank you! Quickly updated the manifest :)

@bleatbot
Copy link
Collaborator

All builds OK!

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

Name Commit Status
✔️ Racingway [testing-live] 18d297f v0.0.1.0 - Repo (New plugin)
1 hidden need (known safe NuGet packages).
Show log - Review

@bleatbot bleatbot removed pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. labels Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. new plugin This is a new plugin. size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants