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.

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

Various enhancements #159

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Various enhancements #159

wants to merge 3 commits into from

Conversation

Xav83
Copy link
Contributor

@Xav83 Xav83 commented Jan 27, 2021

  • Corrects the references to Python version in the example of the documentation
  • Enhances the supports of some CIs job_id variables

@Xav83
Copy link
Contributor Author

Xav83 commented Jan 27, 2021

Hi @eddyxu,

I hope you don't mind me already creating another Pull Request ^^

I am currently using cpp-coveralls for one of my projects, and I would like to help to make it better, for me as for others 🙂
So, if you are fine with it, I will create some others Pull Requests in the near future.

For this Pull Request, I attempted to provide a better the support of Circle CI, as there is an issue on it (#143).
I will probably try to create another Pull Request, for other CIs. To do so, I will inspire myself of node-coveralls project, and more speciffically of this file where the environment variable for each CIs are specified. 😉

@Xav83
Copy link
Contributor Author

Xav83 commented Feb 3, 2021

gentle ping @eddyxu 🙂

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.

1 participant