canonical/charming-actions

Upload to charmhub branches for all cases, not just for git branches named `/branch/...`

Open

#13 opened on Jan 13, 2022

View on GitHub
 (3 comments) (1 reaction) (0 assignees)TypeScript (28 forks)auto 404
Priority: MediumType: Enhancementgood first issue

Repository metrics

Stars
 (9 stars)
PR merge metrics
 (PR metrics pending)

Description

Rather than selectively uploading intermediate work to charmhub branches (eg: latest/edge/someBranch), we could upload all PR updates to charmhub. charmhub branches are ephemeral (ttl=30day) anyway so it won't make too much garbage, and this way we can easily test deployments.

There might be some logic that needs working through as we still need to know the destination track/channel that the branch should go to, but this could be the same as we currently use (if git branch is called /track/..., push to charmhub at /track/edge/branchName, else push to latest/edge/branchname)

Contributor guide

Research direction
Understand the current logic in the action that determines when to upload to charmhub branches. Modify it to always upload for all PR updates, using the same track/channel derivation (if the git branch is named /track/..., push to /track/edge/branchName, else push to latest/edge/branchName). Test the change by creating a sample PR and verifying the upload occurs.
Tech stack
typescript
Domain
devopstoolingci cd
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitGitHub ActionsTypeScript
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65