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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Direct Note for Windows Users (New Comers) #2857

Merged
merged 1 commit into from Feb 10, 2022

Conversation

OsmiumOP
Copy link
Contributor

@OsmiumOP OsmiumOP commented Feb 7, 2022

Signed-off-by: OsmiumOP harshbansal.contact@gmail.com

Fixes #2845
Recreated due to some issues with previous ones.

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 馃搼 and links the original issue above 馃敆
  • Tests pass -- look for a green checkbox 鉁旓笍 a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 馃搧

Still trying to get my First Small PR get merged. Wish me luck and I look forward to advice on how I can improvise.

Signed-off-by: OsmiumOP <harshbansal.contact@gmail.com>
@OsmiumOP OsmiumOP force-pushed the newcomers-docs-update-osmiumop branch from 2895acb to ff2fcc1 Compare February 7, 2022 14:47
@OsmiumOP
Copy link
Contributor Author

OsmiumOP commented Feb 7, 2022

@pombredanne Fixed all issues that existed in the old PR. Please check and let me know if something else is needed.

@pombredanne
Copy link
Member

pytest-dev/pytest#9608 is likely the cause for the failures

@OsmiumOP
Copy link
Contributor Author

OsmiumOP commented Feb 8, 2022

@pombredanne so is there a solution for me to execute? or will you approve and merge as it is?

@the-illuminatus
Copy link

is this issue open ?

@OsmiumOP
Copy link
Contributor Author

OsmiumOP commented Feb 8, 2022

is this issue open ?

The Tests Failing? yes

@pombredanne pombredanne merged commit 3d6434f into nexB:develop Feb 10, 2022
@OsmiumOP OsmiumOP deleted the newcomers-docs-update-osmiumop branch February 10, 2022 18:30
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.

Direct Link to Docs for Windows Users
4 participants