AstrolabeCAE/SiteWebAstrolabe
AstrolabeCAE/SiteWebAstrolabe
2
2
Fork 5
Code Issues 2 Pull Requests Packages Projects Releases 3 Activity

Ajout copyright and licence.txt (copyright est un lien symbolique vers licence) #42

Merged
florent.lesaout merged 7 commits from data-protection-publication into master 2023-03-22 16:45:44 +01:00
Conversation 15 Commits 7 Files Changed 5 +17 -1
florent.lesaout commented 2023-03-11 10:35:50 +01:00
Owner
Copy Link

Prêt pour relcture

Prêt pour relcture
florent.lesaout added 1 commit 2023-03-11 10:35:51 +01:00
florent.lesaout added 1 commit 2023-03-11 10:38:03 +01:00
Ghost commented 2023-03-11 12:36:44 +01:00
First-time contributor
Copy Link

Je serais d'avis de mentionner cette exclusion dans le LICENCE global du projet.
Cf https://opensource.stackexchange.com/a/2695 qui donne un exemple un peu similaire.

Au passage, j'ai l'impression que notre licence n'est même pas valide, elle mentionne un certain andy-bell.design, on le connait ?
On ne mentionne même pas la référence à Eleventy sur lequel on est basé.

MIT License

Copyright (c) 2019 andy-bell.design and other contributors

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

Est-ce qu'on ne pourrait pas modifier le fichier sous la forme

This project is under MIT License for all its code except the src/members subdirectory and its content. 

MIT License

Copyright (c) 2019-2023 Astrolabe CAE

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

It is based on Eleventy whose license can be found below : 
  MIT License

  Copyright (c) 2017–2023 Zach Leatherman @zachleat

  Permission is hereby granted, free of charge, to any person obtaining a copy
  of this software and associated documentation files (the "Software"), to deal
  in the Software without restriction, including without limitation the rights
  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
  copies of the Software, and to permit persons to whom the Software is
  furnished to do so, subject to the following conditions:

  The above copyright notice and this permission notice shall be included in all
  copies or substantial portions of the Software.

  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
  SOFTWARE.

On peut sinon simplement ajouter Zach Leatherman à notre copyright
https://opensource.stackexchange.com/a/5488

This project is under MIT License for all its code except the src/members subdirectory and its content. 

MIT License

Copyright (c) 2019-2023 Astrolabe CAE
Copyright (c) 2017–2023 Zach Leatherman @zachleat

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.
Je serais d'avis de mentionner cette exclusion dans le `LICENCE` global du projet. Cf https://opensource.stackexchange.com/a/2695 qui donne un exemple un peu similaire. Au passage, j'ai l'impression que notre licence n'est même pas valide, elle mentionne un certain andy-bell.design, on le connait ? On ne mentionne même pas la référence à Eleventy sur lequel on est basé. ``` MIT License Copyright (c) 2019 andy-bell.design and other contributors Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. ``` Est-ce qu'on ne pourrait pas modifier le fichier sous la forme ``` This project is under MIT License for all its code except the src/members subdirectory and its content. MIT License Copyright (c) 2019-2023 Astrolabe CAE Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. It is based on Eleventy whose license can be found below : MIT License Copyright (c) 2017–2023 Zach Leatherman @zachleat Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. ``` On peut sinon simplement ajouter Zach Leatherman à notre copyright https://opensource.stackexchange.com/a/5488 ``` This project is under MIT License for all its code except the src/members subdirectory and its content. MIT License Copyright (c) 2019-2023 Astrolabe CAE Copyright (c) 2017–2023 Zach Leatherman @zachleat Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. ```
jeremy.dufraisse requested review from jeremy.dufraisse 2023-03-13 11:34:12 +01:00
jeremy.dufraisse refused to review 2023-03-13 11:34:48 +01:00
jeremy.dufraisse commented 2023-03-13 11:36:21 +01:00
Collaborator
Copy Link

@florent.lesaout J'aimerais bien faire un revue de cette PR mais je ne trouve pas le bouton pour le faire. Est-ce normal ? (si ça vient de moi avec un bouton bien visible, c'est que ça n'est pas normal ;-).. et qu'il me faut nettoyer mes lunettes)

*@florent.lesaout J'aimerais bien faire un revue de cette PR mais je ne trouve pas le bouton pour le faire. Est-ce normal ? (si ça vient de moi avec un bouton bien visible, c'est que ça n'est pas normal ;-).. et qu'il me faut nettoyer mes lunettes)*
Ghost commented 2023-03-13 11:37:08 +01:00
First-time contributor
Copy Link

image

Tu n'as pas ça dans l'onglet fichiers modifiés ?

![image](/attachments/bd1e0b90-987c-4098-8380-4e3dd3788e5e) Tu n'as pas ça dans l'onglet fichiers modifiés ?
image.png
122 KiB
jeremy.dufraisse approved these changes 2023-03-13 11:38:14 +01:00
jeremy.dufraisse left a comment
Collaborator
Copy Link

OK pour moi, merci pour la mise en forme et la vigilance dans la licence MIT

OK pour moi, merci pour la mise en forme et la vigilance dans la licence MIT
jeremy.dufraisse commented 2023-03-13 11:38:59 +01:00
Collaborator
Copy Link

merci @fabien.lefrapper effectivement, il y a bien ce bouton. Ca vient donc de moi, ... trop l'habitude de GitHub qui met un énorme bouton "faire une revue"

merci @fabien.lefrapper effectivement, il y a bien ce bouton. Ca vient donc de moi, ... trop l'habitude de GitHub qui met un énorme bouton "faire une revue"
Ghost commented 2023-03-13 11:39:25 +01:00
First-time contributor
Copy Link

pour info @jeremy.dufraisse les changements proposés n'incluent pas mes suggestions concernant la licence.

Au cas où ton approbation est conditionnée à leur présence, je préfère prévenir 😇

pour info @jeremy.dufraisse les changements proposés n'incluent pas mes suggestions concernant la licence. Au cas où ton approbation est conditionnée à leur présence, je préfère prévenir 😇
jeremy.dufraisse requested changes 2023-03-13 11:41:40 +01:00
jeremy.dufraisse left a comment
Collaborator
Copy Link

Waouh, effectivement, je n'avais pas vu la subtilité, je m'étais auto-persuadé que tu avais fait une PR ou un commit, ... désolé, je corrige mon approbation pour demander l'intégration de ce que tu proposes.
Qui de nous fait le commit ?

Waouh, effectivement, je n'avais pas vu la subtilité, je m'étais auto-persuadé que tu avais fait une PR ou un commit, ... désolé, je corrige mon approbation pour demander l'intégration de ce que tu proposes. Qui de nous fait le commit ?
Ghost commented 2023-03-13 11:44:12 +01:00
First-time contributor
Copy Link

Généralement sur une PR j'émets des commentaires que l'auteur peut ou non prendre en compte.

J'évite de commit directement sur une branche qui ne "m'appartient pas", d'autant plus que Florent n'a visiblement pas validé l'une ou l'autre de mes suggestions.

Je te laisse commit si tu le souhaites, pour ma part je préfère attendre un retour de Florent

Généralement sur une PR j'émets des commentaires que l'auteur peut ou non prendre en compte. J'évite de commit directement sur une branche qui ne "m'appartient pas", d'autant plus que Florent n'a visiblement pas validé l'une ou l'autre de mes suggestions. Je te laisse commit si tu le souhaites, pour ma part je préfère attendre un retour de Florent
jeremy.dufraisse commented 2023-03-13 12:23:30 +01:00
Collaborator
Copy Link

pas de souci @fabien.lefrapper , je sais que je suis du genre à faire des PR sur les branches des autres PR pour faciliter la validation ou non de mes propositions par le porteur de la PR principale...

Et oui, nous avons tous nos habitudes, ... je note pour ton côté. ;-)

pas de souci @fabien.lefrapper , je sais que je suis du genre à faire des PR sur les branches des autres PR pour faciliter la validation ou non de mes propositions par le porteur de la PR principale... Et oui, nous avons tous nos habitudes, ... je note pour ton côté. ;-)
👍 1
Ghost commented 2023-03-13 12:26:06 +01:00
First-time contributor
Copy Link

Oui je sais que ça dépend beaucoup des projets et modes de gouvernance.

J'ai pas particulièrement de chapelle pour être honnête, si on est tous au courant du mode de fonctionnement du projet, et que des contribs à plusieurs sur une même PR ne dérangent personne, tout me va 🤷‍♂️

Oui je sais que ça dépend beaucoup des projets et modes de gouvernance. J'ai pas particulièrement de chapelle pour être honnête, si on est tous au courant du mode de fonctionnement du projet, et que des contribs à plusieurs sur une même PR ne dérangent personne, tout me va 🤷‍♂️
florent.lesaout added 3 commits 2023-03-17 16:55:14 +01:00 
to answer to #42 (comment)
Reviewed-on: #43
Ghost approved these changes 2023-03-17 16:55:52 +01:00
Ghost requested changes 2023-03-17 16:57:05 +01:00
Ghost left a comment
First-time contributor
Copy Link

J'ai approuvé trop vite, il y a une petite coquille dans src/members/LICENCE.txt

J'ai approuvé trop vite, il y a une petite coquille dans src/members/LICENCE.txt
src/members/LICENCE.txt Outdated
@ -0,0 +1,4 @@
Copyright 2023 Astrolabe CAE : All rights reserved
This licence apply to the current folder.
Ghost commented 2023-03-17 16:56:26 +01:00
First-time contributor
Copy Link
This licence applies to the current folder.
``` This licence applies to the current folder. ```
florent.lesaout requested review from jeremy.dufraisse 2023-03-17 16:57:09 +01:00
florent.lesaout requested review from Ghost Team 2023-03-17 16:57:11 +01:00
florent.lesaout commented 2023-03-17 16:58:55 +01:00
Author
Owner
Copy Link

Je ne suis plus sur de tout comprendre. J'ai intégré les modifs de @jeremy.dufraisse .
@fabien.lefrapper Je ne sais pas maintenant si ça te vas en l'état ou si tu souhaite modifier la licence comme proposé (qui me va bien également) ?

Je ne suis plus sur de tout comprendre. J'ai intégré les modifs de @jeremy.dufraisse . @fabien.lefrapper Je ne sais pas maintenant si ça te vas en l'état ou si tu souhaite modifier la licence comme proposé (qui me va bien également) ?
jeremy.dufraisse commented 2023-03-17 16:59:37 +01:00
Collaborator
Copy Link

@florent.lesaout c'est juste une erreur de type apply => applies

@florent.lesaout c'est juste une erreur de type `apply` => `applies`
jeremy.dufraisse approved these changes 2023-03-17 17:00:23 +01:00
jeremy.dufraisse left a comment
Collaborator
Copy Link

super tout me va (avec ou sans correction de l'erreur de type apply => applies)

super tout me va (avec ou sans correction de l'erreur de type `apply` => `applies`)
Ghost commented 2023-03-17 17:00:28 +01:00
First-time contributor
Copy Link

J'ai juste fait un retour sur le verbe apply qui me semble mal conjugé (on utiliserait plutôt applies ici)

L'interface gitea ne permet pas de faire une suggestion de modification

Ce changement était dans ton commit initial, je ne crois pas que Jérémy y ait touché

J'ai juste fait un retour sur le verbe `apply` qui me semble mal conjugé (on utiliserait plutôt `applies` ici) L'interface gitea ne permet pas de faire une suggestion de modification Ce changement était dans ton commit initial, je ne crois pas que Jérémy y ait touché
florent.lesaout added 1 commit 2023-03-17 17:00:46 +01:00
Ghost approved these changes 2023-03-17 17:00:54 +01:00
florent.lesaout commented 2023-03-17 17:04:23 +01:00
Author
Owner
Copy Link

Parfait, je merge en l'état.

Parfait, je merge en l'état.
👍 1 🚀 1
jeremy.dufraisse commented 2023-03-17 17:07:59 +01:00
Collaborator
Copy Link

Petit commentaire post validation.
Nous n'avons pas protégé par la même licence le dossier images/profiles.
Serait-ce une bonne idée de dupliquer ce que nous venons de faire pour ce dossier aussi ?
(ça peut être fait dans un commit direct ou une branche dédiée histoire d'éviter de surcharger la présente PR)

Petit commentaire post validation. Nous n'avons pas protégé par la même licence le dossier `images/profiles`. Serait-ce une bonne idée de dupliquer ce que nous venons de faire pour ce dossier aussi ? (ça peut être fait dans un commit direct ou une branche dédiée histoire d'éviter de surcharger la présente PR)
florent.lesaout commented 2023-03-17 17:19:16 +01:00
Author
Owner
Copy Link

@jeremy.dufraisse si tu te sens de le faire pour ce dossier en direct via un commit ici, ça me va :)

@jeremy.dufraisse si tu te sens de le faire pour ce dossier en direct via un commit ici, ça me va :)
florent.lesaout added 1 commit 2023-03-17 18:13:33 +01:00
florent.lesaout requested review from jeremy.dufraisse 2023-03-17 18:13:49 +01:00
florent.lesaout requested review from Ghost Team 2023-03-17 18:13:51 +01:00
florent.lesaout commented 2023-03-17 18:14:14 +01:00
Author
Owner
Copy Link

J'ai fait la modif qui en effet me semble logique.

J'ai fait la modif qui en effet me semble logique.
jeremy.dufraisse approved these changes 2023-03-18 10:49:26 +01:00
jeremy.dufraisse left a comment
Collaborator
Copy Link

super @florent.lesaout tu as dégainé plus vite que moi (j'étais déconnecté depuis mon dernier commentaire).
Merci pour la réactivité
C'est encore mieux maintenant

super @florent.lesaout tu as dégainé plus vite que moi (j'étais déconnecté depuis mon dernier commentaire). Merci pour la réactivité C'est encore mieux maintenant
florent.lesaout merged commit b9391491af into master 2023-03-22 16:45:44 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Ghost
jeremy.dufraisse
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AstrolabeCAE/SiteWebAstrolabe#42
No description provided.
Delete Branch "data-protection-publication"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?