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

Minor fixes #747

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Minor fixes #747

wants to merge 7 commits into from

Conversation

michaelsauter
Copy link
Member

@michaelsauter michaelsauter commented Mar 8, 2024

See individual commits for more information.

Tasks:

  • Updated design documents in docs/design directory or not applicable
  • Updated user-facing documentation in docs directory or not applicable
  • Ran tests (e.g. make test) or not applicable
  • Updated changelog or not applicable

This breaks tests of tasks that do not need Nexus. Strictly speaking,
Nexus is not needed at all for ODS Pipeline. It is only required when
storing artifacts for later use (e.g. promotion).
docker inspect may also return image information, which is not desired.
@michaelsauter michaelsauter self-assigned this Mar 8, 2024
While helpful in general, this breaks tests of tasks that do not use
Bitbucket. Going forward we should find a way to require the Url but set
it e.g. to https://example.com when tests do not require Bitbucket. For
now, this is just a quick fix to unblock tests from upgrading.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant