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 overriding header while pushing deployment scripts #815

Merged
merged 3 commits into from
Oct 16, 2021

Conversation

christianlupus
Copy link
Collaborator

Currently, the deployment scripts are non-functional as the authorization header is set in the config file and overrides the authorization parameters in the URL. Thus, the authorization headers must be cleared for successfully pushing using a PAT.

Signed-off-by: Christian Wolf <github@christianwolf.email>
Signed-off-by: Christian Wolf <github@christianwolf.email>
@christianlupus christianlupus force-pushed the fix/800-no-push-possible branch from 8bcb601 to 3a724d4 Compare October 16, 2021 16:19
Signed-off-by: Christian Wolf <github@christianwolf.email>
@codecov
Copy link

codecov bot commented Oct 16, 2021

Codecov Report

Merging #815 (3e66153) into master (c1b0e30) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #815   +/-   ##
=======================================
  Coverage   21.40%   21.40%           
=======================================
  Files          20       20           
  Lines        1542     1542           
=======================================
  Hits          330      330           
  Misses       1212     1212           
Flag Coverage Δ
integration 7.13% <ø> (ø)
unittests 14.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@github-actions
Copy link

github-actions bot commented Oct 16, 2021

Unit Test Results

  20 files    20 suites   7m 45s ⏱️
  66 tests   66 ✔️ 0 💤 0
660 runs  660 ✔️ 0 💤 0

Results for commit 3e66153.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus merged commit a3db4e6 into master Oct 16, 2021
@delete-merged-branch delete-merged-branch bot deleted the fix/800-no-push-possible branch October 16, 2021 16:46
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.

1 participant