-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add inline comments on commits #4898
Comments
Sorry, I don't want to be rude nor a smartass, I just want to understand. Is this issue about
which is supposed to be already supported according to https://docs.gitea.io/en-us/comparison/ ? |
@LukeOwlclaw this issue is inline comments on commits. Inline comments on code in PRs already (per the docs you linked). also please take this comment same way, I'm a bit sleepy due to jetlag so I apologize if this response doesn't come off as friendly (I tried to make it friendly). |
@xxxTonixxx is there a PR that currently addresses this? |
I think no 😢 |
@techknowlogick I know open source projects hate asking for ETAs, and I really don't want to come off as rude, but I'm just curious how high on the priority list is this in? There doesn't seem to be a milestone listed, so what are the odds that we'll see this implemented in 1.7.x? Is this something the team is actively planning to work on or is it deferred for sometime later? Just curious so that I can set my expectations clear 😅 |
@rakshith-ravi sadly it won’t be in 1.7 as we are a in a feature freeze for that release, as for ETAs I cant say, as for the project we don’t have a roadmap and things are added to the project as PRs happen. That means even a non-team member could work on this, if you or someone you know has Holland skills. As a side note you don’t come off as rude and I really appreciate how you asked as it was really kind :) |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions. |
Not yet, please 😢 |
Yeah this would be fantastic! |
Vote |
So If we can comment on any commit. We should consider some implementation things below:
|
I'd recommend a structure like this -
If I knew GoLang better, I would've implemented this myself. Unfortunately, I can only contribute in terms of software architecture |
I suspect we should leverage git notes or git notes-like behaviour for this. I suspect that putting this in to the database is only going to cause trouble. I wonder if the comments on PRs should do the same. |
@rakshith-ravi If you can program in an imperative language you can program in go. It's really not a very difficult language if you can get past its idiosyncrasies. |
PR comments have been implemented. I think we are talking about commit comment without a PR. |
Vote. Gitea really lacks this important feature. Everything else I could live on without but not this. |
@0x416c69 Just so I understand your use case, why comments are so important in a specific commit? I've seen GitHub has that, but I never could picture a proper scenario for this. Commits get quickly lost in the sea of blobs.... |
I use this function. Especially to signal to the people who sent the commit that something is wrong, exempting me from opening an issue. It is also useful if I want to add something extra to that commit or mention something. |
We also need refactor notification system to notify via ui/mail/webhook and etc. |
@guillep2k Best way possible for me to talk about a certain commit is the commit comment. Although I could just stick with a messenger and a group talk, place the commit link/hash and talk about it there with my team but it will get really messy and the track of the conversation will be lost very quickly. As a basic and common user of git, this I think is the most needed feature. |
We use it a lot for code reviewing PRs before merging to master.
|
@jcfigueiredo the review functionality already exists in PRs where you can comment on lines of code. |
this "may" also be useful to reference a commit or part of it to an issue without touching issue itself (examples: "may make issue #1 reappear", "this exact part breaks PR !2", etc.) or discussing about a specific part of code. Again, these can already be done the other way around (eg:"commit b822518 may make this issue reappear", "the changes on modules/setting/queue.go lines 58-59 of commit c779ac1 breaks this PR", etc.) |
Even for forgetful programmers, this would be a blessed feature. I swear there are more instances where I forget what a piece of code does than actual files of source code that I have written in my life. I could potentially solve this by writing summaries over my own code. |
IMHO students should first learn how to use proper management of their resources (know what an issue is, why it's filed, its purpose, when to close an issue, PRs and everything related, etc), and then learn git and apply their git knowledge on top of what they're already familiar with workflow-wise. First give the fundations, then build on top. |
+1 very useful feature |
Hi Gitea's team and contributors ❤️, |
@MOHHALIM As this is an Free Software Project: A more constructive question would be: What can we (as a community) do to get this feature implemented soon? E.g. I just realized, that there is the possibility to support the work on open issues on bountysource: https://app.bountysource.com/teams/gitea By searching for this issue here I accidentally created a new bounty: Also, there is the the possibility to sponsor this project (i.e. support in a not issue specific way). |
Like the LibreOffice folks say, "if you want a new feature you suggest it. Then either make it yourself or fund it". |
IMO, we're taking this GoFundMe thing too far. I get it. We all get it. Money doesn't grow on trees, but this is the open source community you guys are talking to. Money doesn't come from any of our pockets. This is the poor programmers club GH issues is for contributors and users to work on features. Not electronically pan-handle each other. Look I'm broke too, but this is just poorly representing the FOSS community (Emphasis on F OSS). The FOSS defacto standard for this is to just tell people "we're too broke to work on it right now," not "please give me money to do it" |
This comment was marked as spam.
This comment was marked as spam.
@KaKi87 That just makes it so much worse. |
Just so it's clear, these comments get sent out to subscribers through their emails. This means that this kind of behavior goes against the Github Acceptable Use Policy. These kind of comments are also considerably off-topic which is also against multiple policies in the Acceptable Use Policies and GIthub Terms of Service. To find out what's on topic for GH Issues please see https://docs.github.com/en/issues/tracking-your-work-with-issues/about-issues |
This comment was marked as outdated.
This comment was marked as outdated.
In which version ? |
Erm… What? And that is not yet supported. |
Is there any current Update to this Feature or? |
💎 $300 bounty • GiteaSteps to solve:
Thank you for contributing to go-gitea/gitea! Add a bounty • Share on socials
|
Personally really interested in this gitea thing so imma give it a shot |
Add comments on commits (not only in PR), I would love to have this feature 😃
Reference: #124, #124 (comment)
Thanks to gitea's team and contributors ❤️
The text was updated successfully, but these errors were encountered: