Skip to content

feature(travis-ci): add basic test to identify issues with configs#2583

Merged
dgibbs64 merged 55 commits intodevelopfrom
feature/tests_defaultcfg
Nov 10, 2019
Merged

feature(travis-ci): add basic test to identify issues with configs#2583
dgibbs64 merged 55 commits intodevelopfrom
feature/tests_defaultcfg

Conversation

@dgibbs64
Copy link
Copy Markdown
Member

Description

added new WIP unit test to confirm config files are correct. Currently, only checks settings and comments that are present in all config files are correct. This will need to be developed more but is a good start

Fixes #[issue]

Type of change

  • Bug fix (change which fixes an issue).
  • New feature (change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

Checklist

PR will not be merged until all steps are complete.

  • This pull request links to an issue.
  • This pull request uses the develop branch as its base.
  • This code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have checked that this code is commented where required.
  • I have provided a detailed enough description of this PR.
  • I have checked If documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

Thank you for your Pull Request!

@dgibbs64 dgibbs64 merged commit 4e5a04e into develop Nov 10, 2019
@delete-merged-branch delete-merged-branch Bot deleted the feature/tests_defaultcfg branch November 10, 2019 22:52
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant