Skip to content

Conversation

@Nasirhus
Copy link
Owner

Please make sure you are raising this PR against your own repository and not the original. If it says https://github.com/CorndelWithSoftwire in the search bar right now, you are in the wrong place!

Please also don't forget to submit the PR's url to Aptem afterwards (by clicking on the corresponding exercise component, pasting the url in the textbox that appears and then pressing the finish button).

@Nasirhus Nasirhus closed this Sep 10, 2024
@Nasirhus Nasirhus reopened this Sep 10, 2024
@Nasirhus
Copy link
Owner Author

please review

if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/Mod9CD' }}
steps:
- uses: actions/checkout@v4
- run: echo hello deploy

Choose a reason for hiding this comment

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

Good practice for clean code is to remove debug messages like this before merging.

@ashleyjkell
Copy link

ashleyjkell commented Sep 12, 2024

Quick comment on clean code but otherwise looks good! It looks like you have some merge conflicts though.

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.

3 participants