Skip to content
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

Any comment about upcoming Git Rev News edition 33 #262

Closed
chriscool opened this issue Nov 14, 2017 · 20 comments
Closed

Any comment about upcoming Git Rev News edition 33 #262

chriscool opened this issue Nov 14, 2017 · 20 comments

Comments

@chriscool
Copy link
Collaborator

A currently mostly empty draft is there:

https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-33.md

Feel free to comment in this issue, or to use the edit button (that looks like a pen) to edit and create a pull request with the changes you would like.

Thanks!

cc @tfnico @jnareb @mjaix @gitster @stefanbeller

@jreybert
Copy link
Contributor

Hi, self-promotion proposition.

You talked several times about the emacs magit plugin, you may want to take a look to its little brother vim plugin vimagit.

It is not as complete as magit, it focuses for the moment to the stage/commit workflow (I intend to add checkout, stash, fixup support in the next year). And for the record, it does not aim to replace fugitive for what it does very well: Ggrep, Gblame, Gedit...

@chriscool
Copy link
Collaborator Author

@jreybert thanks for the suggestion! We will add something about vimagit in this edition.

@dscho
Copy link
Member

dscho commented Nov 17, 2017

@jreybert you could always open a Pull Request ;-) That's what I do when I have to self-promote some of my work... 😜

@chriscool
Copy link
Collaborator Author

Let's publish this edition on Wednesday 22nd November!

@chriscool
Copy link
Collaborator Author

@jreybert vimagit added in 4f23651

@tfnico
Copy link
Contributor

tfnico commented Nov 19, 2017

My links have landed!

@jreybert
Copy link
Contributor

Thanks @chriscool

@jnareb
Copy link
Member

jnareb commented Nov 20, 2017

Do you know when Git for Windows acquired automatic checking for update? If it was in October it would be worth mentioning it in Git Rev News (and even if it was earlier, if there is some blog post about this new feature).

@jrn
Copy link

jrn commented Nov 20, 2017

https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-33.md#support says 'A discussion then started about the merits of having an entry like "*.sh text eol=lf" in the .gitattributes for shell scripts, compared to having Git change strictly no file. It appeared that there is no clear answer about what is best', which makes it sound like the discussion didn't come to any conclusion. But https://public-inbox.org/git/f56a02d6-fbf9-188f-d19f-3d48708d9268@kdbg.org/ looks like a clear conclusion to me.

More generally, I am wondering about the audience for this summary of a support thread. The summary seems to recap each individual message --- maybe there is a way to make it more concise.

@chriscool
Copy link
Collaborator Author

@jrn I was under the impression that https://public-inbox.org/git/alpine.DEB.2.21.1.1710271716330.6482@virtualbox/ could be a different conclusion of the discussion.

My goal with these articles is not to summarize as much as possible, but to tell a story that will hopefully at once do some of the following:

  • interest people,
  • make them learn something about Git,
  • highlight the work by reviewers, user helpers and developers.

@jrn
Copy link

jrn commented Nov 20, 2017

@jrn I was under the impression that https://public-inbox.org/git/alpine.DEB.2.21.1.1710271716330.6482@virtualbox/ could be a different conclusion of the discussion.

Hm, care to elaborate (preferably in that thread :))?

@chriscool
Copy link
Collaborator Author

chriscool commented Nov 21, 2017

Well Dscho points to an example where setting "*.sh text eol=lf" fails to do anything useful in some cases. So I think it shows that the above settings might not be useful in some cases.

@jrn
Copy link

jrn commented Nov 21, 2017

That sounds like a "no, you are not interested in replying there".

shrug

@chriscool
Copy link
Collaborator Author

After thinking about it perhaps the changes in #265 would be good.

And yeah, I am not interested in replying in the thread, because my current goal is not to try to make people change their opinion about what is best, but just to summarize as well as possible the conclusion of the mailing list thread in the Git Rev News article.

@jnareb
Copy link
Member

jnareb commented Nov 21, 2017

My links have landed in 2bcdaa4

@mjaix
Copy link
Contributor

mjaix commented Nov 21, 2017 via email

@mjaix
Copy link
Contributor

mjaix commented Nov 21, 2017 via email

@chriscool
Copy link
Collaborator Author

@jnareb, you should probably ask @dscho about when Git for Windows acquired automatic checking for update.

@chriscool
Copy link
Collaborator Author

Thanks @tfnico, @jnareb and @mjaix for the links and fixes.
@mjaix Torsten's Gerrit related statement looks ok to me, so I left it as is.

@dscho
Copy link
Member

dscho commented Nov 25, 2017

Do you know when Git for Windows acquired automatic checking for update?

Yes I do: v2.14.2 (Sep 26th, see https://github.com/git-for-windows/build-extra/blob/master/ReleaseNotes.md#changes-since-git-for-windows-v2141-august-10th-2017).

Please note that it is an opt-in feature for now, although we may soon switch it to opt-out instead. I'd like the feature to be tested a bit more, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants