assets option is ambiguous with git plugin assets option
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- git, github, javascript
Piste de recherche
Commencez par retracer la façon dont les plugins git et github résolvent l’option assets commune de niveau supérieur et la façon dont les options spécifiques aux plugins la remplacent. Examinez le comportement existant de configuration et de compatibilité avant de décider comment des noms distincts devraient fonctionner. C’est terminé lorsque les assets des commits git et les assets des releases GitHub peuvent être configurés indépendamment, tout en continuant à prendre en charge les configurations assets existantes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Both the git plugin and github plugin have a configuration option named assets. When building out a shareable configuration, It is preferable to omit configuration inline with the plugin and define as much as possible in the top level of the release config so they can be overridden when the configuration is extended.
// my-release-config/index.js
module.exports = {
npmPublish: false,
releaseRules: [...],
assets: ['package.json', '*.md', '!**/node_modules/**'], // <---- assets
changelogTitle: '## Changelog',
plugins: [
['@semantic-release/commit-analyzer', null],
['@semantic-release/release-notes-generator', null],
['@semantic-release/changelog', null]
['@semantic-release/npm', null],
['@semantic-release/git', null],
['@semantic-release/exec', null],
['@semantic-release/github', null]
]
}
This works rather well except when you use the git and github plugins together - the assets configuration is ambiguous meaning. In the above example, All .md files will be included in both the release commit, and the files attached to the github release. if the value needs to be different between the two plugins, you have to define the plugin configuration inline, which makes it difficult to change
// my-release-config/index.js
module.exports = {
npmPublish: false,
releaseRules: [...],
changelogTitle: '## Changelog',
plugins: [
['@semantic-release/commit-analyzer', null],
['@semantic-release/release-notes-generator', null],
['@semantic-release/changelog', null]
['@semantic-release/npm', null],
['@semantic-release/git', {
assets: ['package.json', '*.md', '!**/node_modules/**'] // <---- assets
}]
['@semantic-release/exec', null],
['@semantic-release/github', {
assets: ['dist/*.tgz', 'coverage/*.json'] // <---- assets
}]
]
}
This allows them to be different, But now you cannot really over ride these values individually with out modifying them both, or changing them in the actualy config package being exteded. Which mean people have to re-define the entire plugin change and re-create most of the default configuration shipped with the shared config. Which mostly defeats the purpose.
Ideally, these settings should be named differently to disambiguate them - gitCommitAssets and githubReleaseAssets.
Something to this effect. It could initially be alternate value to allow the existing assets value to work if defined like it currently does.
- Langage dominant
- JavaScript
- Étoiles
- 537
- Forks
- 147
- Merge moyen
- 25 min
- PR mergées (30 j)
- 9
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de semantic-release/github
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
semantic-release/github#1297 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
semantic-release/github#1242 · 1 commentaire · 4 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
semantic-release/github#1103 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
[@semantic-release/github] step fails with 404 error when trying to access non-existent PR # 1Peut-être pris @sjh9714 l’a pris il y a 41 jours. Ouvertebug
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
semantic-release/github#1092 · 6 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
immutable releasesOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
semantic-release/github#1082 · 5 commentaires · 2 réactions ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de semantic-release/github
Issues similaires
-
Four problem loader browser tests fail on WindowsPeut-être pris @Patriciaath999 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
juice-shop/juice-shop#3686 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
ScottKirvan/GOMAC#56 ·
-
area:install bug needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
yetone/magpie#2406 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour