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

Congratulations 1.3.0.0 #4085

Merged
merged 2 commits into from
Jul 20, 2024
Merged

Congratulations 1.3.0.0 #4085

merged 2 commits into from
Jul 20, 2024

Conversation

jkchen2
Copy link
Contributor

@jkchen2 jkchen2 commented Jul 20, 2024

This is a request to adopt the Congratulations plugin, as the original author (Berna) has decided to stop development. I don't intend to do feature development, but at the very least wanted to get this plugin working on Dawntrail.

(This is another plugin from Berna that I am adopting, the first one being the Tf2CriticalHitsPlugin)

Berna's message on Discord regarding stopping development and leaving their plugins up for adoption: https://discord.com/channels/581875019861328007/1069307522269319178/1259198126376488960

Berna's message on GitHub in response to my request to have the plugin be adopted: Berna-L/ffxiv-tf2-crit-plugin#7 (comment)

Additionally, like before, I've elected to keep Berna as the author, as I only intend to keep this plugin compatible with FFXIV/Dalamud updates. If this needs to change, please let me know.

@bleatbot bleatbot enabled auto-merge (squash) July 20, 2024 07:44
@bleatbot
Copy link
Collaborator

bleatbot commented Jul 20, 2024

Outdated attempt

⚠️ No builds attempted! This probably means that your owners property is misconfigured.
The average merge time for plugin updates is currently 17 hours.

Name Commit Status
👽 Congratulations [stable] 8a395e4 Not your plugin
👽 Congratulations [testing-live] 8a395e4 Not your plugin
Show log - Review

@philpax
Copy link
Contributor

philpax commented Jul 20, 2024

Hi! You need to be in the owners array of the TOML to be able to push updates.

auto-merge was automatically disabled July 20, 2024 08:13

Head branch was pushed to by a user without write access

@bleatbot
Copy link
Collaborator

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

Name Commit Status
✔️ Congratulations [stable] 8a395e4 v1.3.0.0 - Diff (43 lines, prev. 1.2.3.0) - Semantic
✔️ Congratulations [testing-live] 8a395e4 v1.3.0.0 - Diff (75 lines, prev. 1.2.2.0) - Semantic
Show log - Review

@bleatbot bleatbot added the size-small Diff for this PR is small. label Jul 20, 2024
@jkchen2
Copy link
Contributor Author

jkchen2 commented Jul 20, 2024

Forgot about that, my bad!

@philpax philpax enabled auto-merge (squash) July 20, 2024 08:27
@philpax philpax merged commit d8930c9 into goatcorp:main Jul 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-small Diff for this PR is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants