conda-forge / conda-forge/conda-forge.github.io

Only build on PRs

Open
#647 11 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
170
Forks
320
Avg merge
2d 10h
Merged PRs (30d)
5

Description

Rebuilding on master takes up extra CI time, since we already successfully built things for them to get merged.

To avoid this we could:
1. build the package in the PR as usuall
1. if the build succeeds push the package to a staging area for the package
1. when the PR is merged move the package from the staging into the main

The main issues that need to be tackled to implement this are:
- [ ] Finding the correct package to move, we may produce a lot of packages with very similar names so picking among them might be an issue (maybe pick the latest)
- [ ] Make a staging area, using a staging account so that attackers can't reveal our tokens.
- [ ] Make something to do the moving, maybe a single Circle job or a webservice listening for merged PRs.
- [ ] Require CI to pass for merging, since we don't build on master we can't merge failing PRs.

ping @scopatz

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.