Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added functionality to remove code after a /*DEV*/ comment #436

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

Conversation

downhiller
Copy link

I can see that back in Nov 2017 @tetrode added a similar PR that never got merged. Hopefully this one will.

(This is my first ever OSS contribution and pull request, so apologies if I've done it wrong...!)

Copy link

@tetrode tetrode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally :-), thanks

@downhiller
Copy link
Author

As an aside, can you educate me - what's the difference between

'/\/\*DEV\*\/.*$/m' (treat dollar sign as end of line rather than end of string)

and

'/\/\*DEV\*\/.*/' (match up to the next newline/end of string)

I think nothing, but both you and @matthiasmullie have used the $ & /m variant, so I assume I'm missing some nuance!

@tetrode
Copy link

tetrode commented Jan 13, 2025 via email

@matthiasmullie matthiasmullie force-pushed the master branch 2 times, most recently from 9ddadb8 to 4c4ecee Compare March 6, 2025 09:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants