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

remove pytest-snail from dev requirements file #2909

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,8 @@ commands:
- run:
name: Preparing environment - Brew
command: |
brew --version
brew untap Homebrew/homebrew-cask-versions
brew update
brew upgrade git
brew install fish
Expand Down Expand Up @@ -98,9 +100,10 @@ commands:
key: -<< pipeline.parameters.cache_key_version >>-win-sys-{{ .Branch }}-<< parameters.py_version >>
- run:
name: Preparing environment - system
no_output_timeout: 20m
command: |
choco install -y --no-progress miniconda3
choco install -y --no-progress openssl openjdk11jre
choco install -y --no-progress openjdk
C:\tools\miniconda3\Scripts\conda.exe init powershell
- run:
name: Preparing environment - Hydra
Expand All @@ -119,7 +122,7 @@ jobs:
py_version:
type: string
macos:
xcode: "13.4.1"
xcode: "15.3.0"
steps:
- macos:
py_version: << parameters.py_version >>
Expand Down
4 changes: 2 additions & 2 deletions noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,13 +87,13 @@ def print_installed_package_version(session: Session, package_name: str) -> None
pip_list: str = session.run("pip", "list", silent=True)
for line in pip_list.split("\n"):
if package_name in line:
print(f"Installed {package_name} version: {line}")
logger.info(f"Installed {package_name} version: {line}")


def install_hydra(session: Session, cmd: List[str]) -> None:
# needed for build
session.install("read-version", silent=SILENT)
# clean install hydra
# clean-install hydra
session.chdir(BASE)
if USE_OMEGACONF_DEV_VERSION:
session.install("--pre", "omegaconf", silent=SILENT)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@

def _run_command(command: str) -> str:
print(f"{str(datetime.now())} - Running: {command}")
output = subprocess.getoutput(command)
output = subprocess.getoutput(command) # nosec B605
print(f"{str(datetime.now())} - {output}")
return output

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

def _run_command(command: str) -> str:
print(f"{str( datetime.now() )} - OUT: {command}")
output = subprocess.getoutput(command)
output = subprocess.getoutput(command) # nosec B605
print(f"{str( datetime.now() )} - OUT: {output}")
return output

Expand Down
2 changes: 1 addition & 1 deletion plugins/hydra_ray_launcher/tests/test_ray_aws_launcher.py
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@

def run_command(commands: str) -> str:
log.info(f"running: {commands}")
output = subprocess.getoutput(commands)
output = subprocess.getoutput(commands) # nosec B605
log.info(f"outputs: {output}")
return output

Expand Down
1 change: 0 additions & 1 deletion requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ packaging
pre-commit
pytest
pytest-benchmark
pytest-snail
read-version
setuptools
towncrier
Expand Down
Loading