Skip to content

Commit

Permalink
Fixed bedset on befile
Browse files Browse the repository at this point in the history
  • Loading branch information
khoroshevskyi committed Jun 10, 2024
1 parent 25c8277 commit e912116
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/cli-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ jobs:
- run: smokeshow upload htmlcov
env:
SMOKESHOW_GITHUB_STATUS_DESCRIPTION: Coverage {coverage-percentage}
SMOKESHOW_GITHUB_COVERAGE_THRESHOLD: 50
SMOKESHOW_GITHUB_COVERAGE_THRESHOLD: 35
SMOKESHOW_GITHUB_CONTEXT: coverage
SMOKESHOW_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SMOKESHOW_GITHUB_PR_HEAD_SHA: ${{ github.event.pull_request.head.sha }}
Expand Down
8 changes: 7 additions & 1 deletion bbconf/config_parser/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,13 @@ class ConfigS3(BaseModel):
@field_validator("aws_access_key_id", "aws_secret_access_key")
def validate_aws_credentials(cls, value):
# Do this if AWS credentials are not provided
if value in ["AWS_SECRET_ACCESS_KEY", "AWS_ACCESS_KEY_ID", ""]:
if value in [
"AWS_SECRET_ACCESS_KEY",
"AWS_ACCESS_KEY_ID",
"",
"$AWS_ACCESS_KEY_ID",
"$AWS_SECRET_ACCESS_KEY",
]:
return None
return value

Expand Down
6 changes: 3 additions & 3 deletions bbconf/modules/bedfiles.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,9 +127,9 @@ def get(self, identifier: str, full: bool = False) -> BedMetadata:
for relation in bed_object.bedsets:
bed_bedsets.append(
BedSetMinimal(
id=relation.bedfile.id,
description=relation.bedfile.description,
name=relation.bedfile.name,
id=relation.bedset.id,
description=relation.bedset.description,
name=relation.bedset.name,
)
)

Expand Down

0 comments on commit e912116

Please sign in to comment.