Skip to content

Enable GitHub actions #10

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

Closed
wants to merge 1 commit into from

Conversation

miraculixx
Copy link
Contributor

@miraculixx miraculixx commented May 23, 2022

closes #2

notes

  • add github actions (you may need to enable on the repo for actions to run)
  • fix issue in TestProxy that failed failed some tests

@@ -130,3 +130,8 @@ dmypy.json

# Project specific
configurable_http_proxy/version.txt
/.idea/configurable-http-proxy.iml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should just make a separate section for # Editors and ignore the entire /.idea folder

python -m pip install --upgrade pip
pip install flake8 pytest
pip install -r requirements.txt
pip install -e .
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-r requirements.txt is not required as the -e . will already install those dependencies

@AbdealiLoKo
Copy link
Collaborator

Wanted to make some changes based on the comments I added, so I have moved this to #12

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.

Setup CI with github actions
2 participants