Skip to content

Conversation

@ernilambar
Copy link
Member

@ernilambar ernilambar commented Mar 20, 2024

Fixes: #217

  • Add github option for --ci argument in scaffold package-tests
  • Two files are added for this CI (apart from common test files).
    • behat.yml - Same file from the root is copied
    • testing.yml - Template file is kept at dot-github/workflows folder. This will be copied to .github/workflows

@ernilambar ernilambar marked this pull request as ready for review March 20, 2024 10:01
@ernilambar ernilambar requested a review from a team as a code owner March 20, 2024 10:01
Copy link

@wojtekn wojtekn left a comment

Choose a reason for hiding this comment

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

The change looks clear, and works fine:

% vendor/bin/wp scaffold package-tests wp-cli-shim --ci=github 
Success: Created package test files.

@danielbachhuber danielbachhuber self-requested a review March 21, 2024 08:07
Copy link
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

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

Can you add some basic tests for this?

@ernilambar
Copy link
Member Author

@danielbachhuber PR has been updated.

@danielbachhuber danielbachhuber self-requested a review March 23, 2024 10:40
Copy link
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

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

Thanks @ernilambar !

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support GitHub Actions CI config

3 participants