Closed ranocha closed 1 month ago
Hello @ranocha,
Thank you very much for your review! I pushed some changes to the paper branch and addressed most of the points you mentioned. Please let me know if you have further comments on these changes.
I have some questions regarding the open points.
Regarding the first bullet point, Kerstin Weinberg provided extended guidance and support throughout the development of the Julia package, offering regular feedback on its direction and plans. Extended pair programming sessions were also used, enhancing our debugging approach and overall code quality. Is adding an author's contribution section to the paper necessary to clarify this aspect, or is this clarification just needed for the review process?
I checked the link in SECURITY.md with different machines, and it worked for me. I compared it with the link in https://github.com/trixi-framework/Trixi.jl/blob/5c9332b944fb37d2432ea9e14c7d8fc77ce8ee24/SECURITY.md?plain=1#L18, and it has the same URL structure. On which browser/device did you check the link?
Regarding the first bullet point, Kerstin Weinberg provided extended guidance and support throughout the development of the Julia package, offering regular feedback on its direction and plans. Extended pair programming sessions were also used, enhancing our debugging approach and overall code quality. Is adding an author's contribution section to the paper necessary to clarify this aspect, or is this clarification just needed for the review process?
It's required for the review process. Your answer is fine with me - that's why I have suggested several options to justify this. It's just part of the (JOSS) review process, which is a bit different from standard journals (due to the focus on code).
I checked the link in SECURITY.md with different machines, and it worked for me. I compared it with the link in https://github.com/trixi-framework/Trixi.jl/blob/5c9332b944fb37d2432ea9e14c7d8fc77ce8ee24/SECURITY.md?plain=1#L18, and it has the same URL structure.
Did you try that while not being logged-in as an admin of this repo?
Please remember to release a new version of the software including your recent improvements - and to merge/rebase your paper branch accordingly so that it contains the full software (see https://joss.readthedocs.io/en/latest/submitting.html#submission-requirements)
I checked the link in SECURITY.md with different machines, and it worked for me. I compared it with the link in https://github.com/trixi-framework/Trixi.jl/blob/5c9332b944fb37d2432ea9e14c7d8fc77ce8ee24/SECURITY.md?plain=1#L18, and it has the same URL structure.
Did you try that while not being logged-in as an admin of this repo?
Yes, within a private browser window and on different machines. Then it is necessary to sign in to submit a security vulnerability.
Please remember to release a new version of the software including your recent improvements - and to merge/rebase your paper branch accordingly so that it contains the full software (see https://joss.readthedocs.io/en/latest/submitting.html#submission-requirements)
Thank you for this reminder! I will submit a patch release with all changes after #167 is fixed.
I still get
Hm, that is really interesting...š§
I have just tested it again with 3 of my colleagues on Linux, Windows and Mac and it worked for them.
If it has anything to do with it, I also have enabled āPrivate vulnerability reportingā and some other advanced settings, maybe that solved it for you.
Please check again, and also check if the green button "Report a vulnerability" is working. And please let me know which URL this button has on your machine, because this should be the same as in SECURITY.md
.
I also have enabled āPrivate vulnerability reportingā
This was it - now it works for me as well
I'll close this in favor of #167
This is part of the review https://github.com/JuliaCon/proceedings-review/issues/165
Thanks a lot for your contributions š Please find below some comments based on the review checklist: