KEMBAR78
Add check_output function for integration tests by Anthuang · Pull Request #305 · dotenv-linter/dotenv-linter · GitHub
Skip to content

Conversation

@Anthuang
Copy link
Contributor

@Anthuang Anthuang commented Oct 3, 2020

This PR adds a check_output function for integration tests, and refactors current tests to use it. The added function is made to be similar to fix_output, which already exists.

Resolves: #293

✔ Checklist:

- [ ] This PR has been added to CHANGELOG.md (at the top of the list);
- [ ] Tests for the changes have been added (for bug fixes / features);
- [ ] Docs have been added / updated (for bug fixes / features).

Crossed out the above since this is just an internal improvement. Let me know if there's anything else I should do!

@mgrachev
Copy link
Member

mgrachev commented Oct 5, 2020

@Anthuang 👋 Thank you for your contribution! 🚀

Please add this PR to CHANGELOG.md (at the top of the Changed list).

@mgrachev mgrachev merged commit f4d8cb3 into dotenv-linter:master Oct 5, 2020
@mgrachev
Copy link
Member

mgrachev commented Oct 5, 2020

@Anthuang Thank you! 👍

🙏 If it’s not difficult for you, please support the project - click on the star on GitHub ⭐️

@Anthuang
Copy link
Contributor Author

Anthuang commented Oct 5, 2020

I'd be happy to! I'm just starting out in Rust and am looking for projects to contribute to. It seems like all the current issues are taken, is there anything else I can help out with?

@mgrachev
Copy link
Member

mgrachev commented Oct 5, 2020

I'd be happy to! I'm just starting out in Rust and am looking for projects to contribute to. It seems like all the current issues are taken, is there anything else I can help out with?

You can take this one #268 😉

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Add check_output function for integration tests

2 participants