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.

Sign up for GitHub

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

Jump to bottom

SNOW-1825476 Implement programmatic access token (PAT) #1298

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

SNOW-1825476 Implement programmatic access token (PAT) #1298

wants to merge 3 commits into from
+358 −28

Conversation

sfc-gh-pfus
Copy link
Collaborator

@sfc-gh-pfus sfc-gh-pfus commented Feb 3, 2025 •
edited
Loading

Description

SNOW-1825476 Implemented PAT/programmatic access token support.

Checklist

  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary

@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-1825476-pat branch 12 times, most recently from 8c3e25f to 23bd952 Compare February 3, 2025 13:08
@sfc-gh-pfus sfc-gh-pfus marked this pull request as ready for review February 3, 2025 13:09
@sfc-gh-pfus sfc-gh-pfus requested a review from a team as a code owner February 3, 2025 13:09
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-1825476-pat branch 8 times, most recently from fa0c6a4 to 0e3f6c4 Compare February 4, 2025 11:16
sfc-gh-pmotacki
sfc-gh-pmotacki reviewed Feb 4, 2025
View reviewed changes
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-1825476-pat branch 2 times, most recently from ffeefdb to d93d803 Compare February 5, 2025 08:26
sfc-gh-dprzybysz
sfc-gh-dprzybysz reviewed Feb 7, 2025
View reviewed changes
@codecov Codecov
Copy link

codecov bot commented Feb 20, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 90.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 82.05%. Comparing base (6988c5f) to head (c4cf34e).

Files with missing lines Patch % Lines
auth.go 89.47% 2 Missing ⚠️
dsn.go 84.61% 2 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1298      +/-   ##
==========================================
+ Coverage   82.04%   82.05%   +0.01%     
==========================================
  Files          55       55              
  Lines       13670    13707      +37     
==========================================
+ Hits        11215    11247      +32     
- Misses       2455     2460       +5

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfc-gh-pmotacki sfc-gh-pmotacki sfc-gh-pmotacki left review comments

@sfc-gh-dprzybysz sfc-gh-dprzybysz sfc-gh-dprzybysz approved these changes

@sfc-gh-dheyman sfc-gh-dheyman sfc-gh-dheyman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sfc-gh-pfus @sfc-gh-dprzybysz @sfc-gh-pmotacki @sfc-gh-dheyman