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

DailyDuty v5.2.0.1 #4928

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

DailyDuty v5.2.0.1 #4928

wants to merge 1 commit into from

Conversation

MidoriKami
Copy link
Contributor

Potentially fix Raids modules (untested)

Remove Teleporter Plugin dependency.

@bleatbot bleatbot enabled auto-merge (squash) November 18, 2024 05:02
@bleatbot
Copy link
Collaborator

bleatbot commented Nov 18, 2024

Outdated attempt

Builds failed, please check action output.

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

The average merge time for plugin updates is currently 2 hours.

Name Commit Status
DailyDuty [stable] 4d04b14 Build failed (Diff)
14 Needs (⚠️ 1 UNREVIEWED)
Type Name Version Reviewed by
Submodule https://github.com/MidoriKami/KamiLib bd70e791bd1d1afa84e3fb9ef12c559c52ded241 ⚠️ Upd. from 427d7919b1f5a154daf6815216086604f804f1b5
Submodule https://github.com/MidoriKami/KamiToolKit.git 8027b76b17f649ccd379e14598858d60dd6a9d42 goaaats
NuGet NetStone 1.1.1 reiichi001
NuGet HtmlAgilityPack 1.11.46 reiichi001
NuGet Karashiiro.HtmlAgilityPack.CssSelectors.NetCoreFork 0.0.2 reiichi001
NuGet Microsoft.Build.Tasks.Git 1.1.1 goaaats
8 hidden needs (known safe NuGet packages).
Show log - Review

@bleatbot bleatbot added the build failed This plugin failed to build. label Nov 18, 2024
@philpax
Copy link
Contributor

philpax commented Nov 18, 2024

bleatbot, rebuild

@bleatbot
Copy link
Collaborator

Builds failed, please check action output.

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

Name Commit Status
DailyDuty [stable] 4d04b14 Build failed (Diff)
14 Needs (✅ All reviewed)
Type Name Version Reviewed by
Submodule https://github.com/MidoriKami/KamiLib bd70e791bd1d1afa84e3fb9ef12c559c52ded241 reiichi001
Submodule https://github.com/MidoriKami/KamiToolKit.git 8027b76b17f649ccd379e14598858d60dd6a9d42 goaaats
NuGet NetStone 1.1.1 reiichi001
NuGet HtmlAgilityPack 1.11.46 reiichi001
NuGet Karashiiro.HtmlAgilityPack.CssSelectors.NetCoreFork 0.0.2 reiichi001
NuGet Microsoft.Build.Tasks.Git 1.1.1 goaaats
8 hidden needs (known safe NuGet packages).
Show log - Review

@philpax
Copy link
Contributor

philpax commented Nov 18, 2024

@goaaats looks like this one will need your intervention as well

@MidoriKami
Copy link
Contributor Author

MidoriKami commented Nov 18, 2024

DailyDuty, NoTankYou, and CurrencyAlert, all use the same combination of KamiLib, KamiToolkit, and main plugin code. It's very odd to me that it works sometimes, but not others.

I won't really have any time until this upcoming weekend to mess with things, so ideally by then we can work something out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build failed This plugin failed to build.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants