Skip to content

Workflow#5665

Open
pgjones wants to merge 1 commit into
mainfrom
workflow
Open

Workflow#5665
pgjones wants to merge 1 commit into
mainfrom
workflow

Conversation

@pgjones

@pgjones pgjones commented Jan 2, 2025

Copy link
Copy Markdown
Member

No description provided.

@pgjones pgjones force-pushed the workflow branch 3 times, most recently from 145f61f to 4044716 Compare January 2, 2025 21:17
@davidism

davidism commented Jan 2, 2025

Copy link
Copy Markdown
Member

It doesn't seem to cut down on much boilerplate, there's still approximately the same amount of workflow code to keep synced across projects. Is there more than can be done?

@pgjones

pgjones commented Jan 3, 2025

Copy link
Copy Markdown
Member Author

Yea, I can also change the pre-commit and publish workflows to use shared actions. If we like this pattern?

@pgjones

pgjones commented Jan 5, 2025

Copy link
Copy Markdown
Member Author

Added pre-commit and build actions - will clean up and fix if we like this pattern.

Comment thread src/flask/sansio/sessions.py Outdated
@@ -0,0 +1,215 @@
from __future__ import annotations

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.

This stuff isnt meant to be in the PR, right?

samuelpondugala

This comment was marked as resolved.

@ThiefMaster

Copy link
Copy Markdown
Member

@samuelpondugala This is not a playground. If you are running bots to auto-comment on github, test them on your own repos.

This should allow for a large reduction in duplicated action code
across the pallet's repos.
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.

5 participants