Skip to content

Allow for manual runs of SF CLI Integration Test#104

Merged
markdlv-sf merged 4 commits into
mainfrom
allow-manual-sf-cli-integ-test-runs
May 29, 2026
Merged

Allow for manual runs of SF CLI Integration Test#104
markdlv-sf merged 4 commits into
mainfrom
allow-manual-sf-cli-integ-test-runs

Conversation

@markdlv-sf
Copy link
Copy Markdown
Contributor

No description provided.


on:
pull_request:
workflow_dispatch:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This allows us to run this github action on main, manually in the github UI.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"Us" being any user with write access to the repo.

Comment thread pyproject.toml
enable = true
pattern = "^v(?P<base>.+)$"
format-jinja = "{% if distance == 0 %}{{ base }}{{ stage or '' }}{% else %}{{ bump_version(base) }}.dev{{ distance }}+{{ commit }}{% endif %}"
pattern = "^v(?P<base>\\d+\\.\\d+\\.\\d+)(?P<stage>\\.dev\\d+|a\\d+|b\\d+|rc\\d+)?$"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The problem we're solving here is when installing/running locally, if the base branch was .dev1, then .post... would be appended to it and that wouldn't match pep440. This makes it so, if you're already working on a tag (like locally), it just uses that instead of trying to add anything.

Comment thread CONTRIBUTING.md
```

4. **Initialize a project for development work verification**
4. **Install the SF CLI and plugin**
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I realize this is getting ahead of the README itself, but I started adding the poetry build instructions (below) as a side effect and realized this really needs to be our main development path- not a side note.


on:
pull_request:
workflow_dispatch:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"Us" being any user with write access to the repo.

@markdlv-sf markdlv-sf merged commit 25507df into main May 29, 2026
3 checks passed
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.

2 participants