Skip to content

Various changes#2058

Draft
Croebh wants to merge 4 commits into
nightlyfrom
jokeen/various-changes
Draft

Various changes#2058
Croebh wants to merge 4 commits into
nightlyfrom
jokeen/various-changes

Conversation

@Croebh

@Croebh Croebh commented Jul 23, 2024

Copy link
Copy Markdown
Contributor

Summary

Gives a more helpful error for DCv1 import issues
When the user gives an ambigous -t target, clarify if some of the options are Groups or not
Update the regex for !campaign links
Don't import attack descriptions that are straight up "If you're proficient, add proficiency!"

Changelog Entry

Here goes a short one line about the PR to display in the Changelog ( optional )

Checklist

PR Type

  • This PR is a code change that implements a feature request.
  • This PR fixes an issue.
  • This PR adds a new feature that is not an open feature request.
  • This PR is not a code change (e.g. documentation, README, ...)

Other

  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • If code changes were made then they have been tested.
  • I have updated the documentation to reflect the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant