Skip to content

Comments

CI Pipeline Runs example/example.py#151

Merged
MOmarMiraj merged 17 commits intomainfrom
morgan/150/example-test
Apr 11, 2025
Merged

CI Pipeline Runs example/example.py#151
MOmarMiraj merged 17 commits intomainfrom
morgan/150/example-test

Conversation

@moward98
Copy link
Contributor

@moward98 moward98 commented Jan 30, 2025

Altered logic in example/example.py to read the previously hardcoded vault_id from environment variables.

Added example/example.py to the validation workflow, and read OP_SERVICE_ACCOUNT_TOKEN and OP_VAULT_ID from GitHub secrets.

Altered the example client.secrets.resolve to resolve field from item created during the example.py item CRUD.

…ith parameters of created item. example.py reads vault id from env var. Call to run example.py added to pipeline
@moward98 moward98 self-assigned this Jan 30, 2025
@github-actions
Copy link

github-actions bot commented Mar 3, 2025

⚠️ This PR contains unsigned commits. To get your PR merged, please sign those commits (git rebase --exec 'git commit -S --amend --no-edit -n' @{upstream}) and force push them to this branch (git push --force-with-lease).

If you're new to commit signing, there are different ways to set it up:

Sign commits with gpg

Follow the steps below to set up commit signing with gpg:

  1. Generate a GPG key
  2. Add the GPG key to your GitHub account
  3. Configure git to use your GPG key for commit signing
Sign commits with ssh-agent

Follow the steps below to set up commit signing with ssh-agent:

  1. Generate an SSH key and add it to ssh-agent
  2. Add the SSH key to your GitHub account
  3. Configure git to use your SSH key for commit signing
Sign commits with 1Password

You can also sign commits using 1Password, which lets you sign commits with biometrics without the signing key leaving the local 1Password process.

Learn how to use 1Password to sign your commits.

Watch the demo

@moward98 moward98 force-pushed the morgan/150/example-test branch 3 times, most recently from df40bab to 337ab76 Compare March 3, 2025 15:58
@moward98 moward98 force-pushed the morgan/150/example-test branch from 337ab76 to 03e4e6e Compare March 3, 2025 16:41
Copy link
Contributor

@MOmarMiraj MOmarMiraj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed up with Morgan and LGTM!

@MOmarMiraj MOmarMiraj merged commit 410d182 into main Apr 11, 2025
21 checks passed
@MOmarMiraj MOmarMiraj deleted the morgan/150/example-test branch April 11, 2025 15:31
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.

3 participants