readthedocs-fr / bin-server

Un outil pour héberger des snippets de code et les partager via une URL.
https://bin.readthedocs.fr
MIT License
14 stars 15 forks source link

ci: release-please + pypi publish #124

Closed fusetim closed 3 years ago

Julien00859 commented 3 years ago

Je te demande ton avis @AntoineJT vu que tu as pas mal bossé sur ces fichiers, si tu es d'accord, si tu as des questions

fusetim commented 3 years ago

Pensez à rename la PR et au "Closes #121"

Mesteery commented 3 years ago

On pourrait profiter de cette PR pour upload sur la release au lieu d'upload dans l'artifact (celui-ci peut expirer).

-    - name: Upload tar.gz
-      uses: actions/upload-artifact@v2.2.1
-      with:
-        name: bin-sources
-        path: dist/bin-${{ env.BIN_VERSION }}.tar.gz
-    - name: Upload wheel
-      uses: actions/upload-artifact@v2.2.1
-      with:
-        name: bin-wheel
-        path: dist/bin-${{ env.BIN_VERSION }}-py3-none-any.whl

+    - name: Upload tar.gz and wheel
+      uses: svenstaro/upload-release-action@v2
+      with:
+        file: dist/*
+        file_glob: true
+        tag: ${{ github.ref }}
+        repo_token: ${{ secrets.GITHUB_TOKEN }}
Mesteery commented 3 years ago

Il faudra aussi penser à réactiver la section Release quand cette PR sera merge.

Mesteery commented 3 years ago

any news ?

fusetim commented 3 years ago

Si on est vraiment pointilleux, faudra changer "Closes #121" par "Resolves #121, resolves #112".

Par ailleurs, je me suis assuré que les workflows étaient valides, mais je ne peux pas les tester sans passer sur #main à cause des EnvVar ¯_(ツ)_/¯

Mesteery commented 3 years ago

Si on est vraiment pointilleux, faudra changer "Closes #121" par "Resolves #121, resolves #112"

Resolves et Closes font la même chose.

fusetim commented 3 years ago

Si on est vraiment pointilleux, faudra changer "Closes #121" par "Resolves #121, resolves #112"

Resolves et Closes font la même chose.

En effet, mais je trouve plus approprié "Resolves" pour plusieurs issues

Julien00859 commented 3 years ago

Bon aller, je squash le commentaire et je merge ça :)

Julien00859 commented 3 years ago

UP vous avez oublié de corriger le fichier MANIFEST.in également https://github.com/readthedocs-fr/bin-server/blob/d75e2841aa40cc13e92666b81b394db2b659805b/MANIFEST.in#L1