scipp / scipp/copier_template

Remove scipp and ess specifics

Open
#196 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Jinja
Stars
2
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Recently some ess specifics were re-added to the repo, see https://github.com/scipp/scippneutron/issues/476 and https://github.com/scipp/copier_template/pull/192#discussion_r1671533053.

We would like to move those specifics to https://github.com/scipp/ess_template if possible, or at least only include them conditionally, if the repo is a scipp- repo or an ess- repo.

One solution would be to make the workflows jinja templates, and make them render the environment variable settings only if the repo is a scipp repo. However, there are some issues with using templates because they are not picked up well by github bots (please correct / clarify this).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing issue #476 and the discussion on copier_template PR #192, then inspect the repository's workflow files for the re-added ESS and Scipp-specific settings. Clarify whether the settings should move to ess_template or be conditional, and verify the chosen approach remains usable by GitHub bots. Done means the unwanted specifics are no longer unconditionally included.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions
Domain
ci-cd
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.