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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dw/regex #26

Merged
merged 4 commits into from
Oct 20, 2022
Merged

Dw/regex #26

merged 4 commits into from
Oct 20, 2022

Conversation

D-Town44
Copy link
Contributor

@D-Town44 D-Town44 commented Oct 19, 2022

馃摉 Description and reason

Add Regex functionality to excluded_paths option

馃懛 Work done

Tasks

  • Test Run
  • Task 2

Additional notes

Also updated the version/readme/tests

馃帀 Result

image

馃 Dispatch

#dispatch/elixir

@D-Town44
Copy link
Contributor Author

Added Regex Functionality. Updated Version/Readme/Tests

@remi
Copy link
Member

remi commented Oct 19, 2022

Hi Daniel,

Could you exclude the version/changelog update from your PR. We like to keep actual feature code separate from version changes.

Thank you!

@D-Town44
Copy link
Contributor Author

No problem! Removed 馃憤

@remi remi self-requested a review October 19, 2022 20:12
@remi remi merged commit 1073124 into mirego:master Oct 20, 2022
@remi
Copy link
Member

remi commented Oct 20, 2022

Thank you @D-Town44! I just released v2.0.1 with your feature 馃帀

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.

None yet

2 participants