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

Fix up all artifactory paths #1290

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

harels
Copy link
Member

@harels harels commented Nov 14, 2022

changing all artifactory urls to the same account

Signed-off-by: Harel Shein [email protected]

Problem

In a previous PR, artifactory paths were changes in the build CI script, but not on all paths in the repo.
Closes: #1237

Solution

This PR changes all artifactory paths to use the same account.

Checklist

  • You've signed-off your work
  • Your pull request title follows our guidelines
  • Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
  • You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
  • You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2022 contributors to the OpenLineage project

changing all artifactory urls to the same account

Signed-off-by: Harel Shein <[email protected]>
@harels harels changed the title Fix up all artifactory paths to astronomer Fix up all artifactory paths Nov 14, 2022
@harels harels requested a review from mobuchowski November 14, 2022 14:38
@harels harels merged commit 4c6930e into OpenLineage:main Nov 14, 2022
@howardyoo
Copy link
Contributor

Awesome! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Circle CI/CD] 401 Unauthorized error when publishing snapshot to JFrog Artifactory
3 participants