Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[#7] Environment variable as testrail config #20

Merged
merged 1 commit into from
Mar 28, 2020
Merged

[#7] Environment variable as testrail config #20

merged 1 commit into from
Mar 28, 2020

Conversation

prashanth-sams
Copy link
Owner

@prashanth-sams prashanth-sams commented Mar 27, 2020 •
edited
Loading

Added simple condition to accept config through Environment variables

  • no changes to be made for the existing users
  • config is made two way: env. variable as well as default value in the config file
      url: ENV['URL']
      user: your@mail.com
      password: ENV['TESTRAIL_USER']
      run_id: ENV['RUN_ID']
      clean_testrun: false
      project_id: 10
      suite_id: 110

@Ilia-ab I really thank you for the awesome #1 pull request, which I took as a base to do this. I just changed the way it was written to cover the above two scenarios.

@prashanth-sams prashanth-sams changed the title [#7] testrail config from cli env [#7] testrail config as environment variable Mar 27, 2020
@prashanth-sams prashanth-sams changed the title [#7] testrail config as environment variable [#7] Environment variable as testrail config Mar 27, 2020
@prashanth-sams
Copy link
Owner Author

@Ilia-ab can you please CR this pull request :)

@prashanth-sams prashanth-sams merged commit dc48ca1 into master Mar 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@prashanth-sams