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

[Testing] Scout Helper v1.7.0 #4583

Merged
merged 1 commit into from
Sep 18, 2024
Merged

[Testing] Scout Helper v1.7.0 #4583

merged 1 commit into from
Sep 18, 2024

Conversation

dit-zy
Copy link
Contributor

@dit-zy dit-zy commented Sep 16, 2024

this change doesn't introduce any new functionality. rather, it migrates a lot of functionality to a helper lib (XIVHuntUtils) that now provides the same capabilities.

XIVHuntUtils (xhu) was created because several features of scout helper are interesting to some other plugins. rather than recreating those features in the other plugins, i created xhu to house those features, and basically copied those things right out of scout helper.

this change also introduces the use of the DitzyExtensions library. that is another library in the vain of xhu -- a bunch of common c# extensions and utils that i ended up creating in scout helper but found to be useful in other c# projects of mine so made a lib.

@bleatbot bleatbot enabled auto-merge (squash) September 16, 2024 22:00
@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!

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

✔️ScoutHelper [testing-live] - c2bba25 - v1.7.0.0 - Diff (90 lines, prev. 1.6.3.0) - Semantic

Show log - Review

@bleatbot bleatbot added the size-small Diff for this PR is small. label Sep 16, 2024
@bleatbot bleatbot merged commit 1ab71ac into goatcorp:main Sep 18, 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