Skip to content

Multiple search and replaces - #25

Open
pedem wants to merge 6 commits into
jacobtomlinson:masterfrom
pedem:master
Open

Multiple search and replaces#25
pedem wants to merge 6 commits into
jacobtomlinson:masterfrom
pedem:master

Conversation

@pedem

@pedem pedem commented Aug 27, 2021

Copy link
Copy Markdown

The keys are not case-sensitive.
Instead of
find: original
replace: new

Just use
include: pattern
exclude: pattern
HOST: "my.domain.com"

In order to get this to run, you'll have to modify the actions.yml and allow all inputs.... if possible.

@jacobtomlinson jacobtomlinson left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for raising this @pedem.

I like the idea here, enabling multiple find and replaces would be awesome. However I'm not sure on the implementation. I don't think it will be very clear and may be error prone. I worry about mixing user input with reserved keywords.

I wonder if we could use some additional nesting.

with:
  include: "test/"
  matrix:
    FOO: "red"
    BAR: "green"
    BAZ: "blue"

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.

2 participants