Skip to content

Drop deprecated git status --show-ignored-directory option #2067

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

Merged
merged 2 commits into from
May 16, 2022

Conversation

dscho
Copy link
Member

@dscho dscho commented Feb 12, 2019

In the v2.15.x timeframe, that option was integrated into core Git, but with a different syntax: --ignored=matching. In Git for Windows, since we offered --show-ignored-directory in at least one official version, we reintroduced it as a backstop, deprecating it and intending to remove it soon.

The time has come.

@dscho
Copy link
Member Author

dscho commented Feb 12, 2019

@clboles Could you please make sure that this option is indeed not used any longer in any projects you maintain?

@jamill
Copy link

jamill commented Feb 12, 2019

This option is still used by the version of Git that ships with Visual Studio 2017. As VFSForGit overrides the version of Git that Visual Studio uses to the one that VFSForGit installs, this will cause issue with VFSForGit and Visual Studio 2017 installed on the same machine.

We could look at moving this patch into only the Git for VFSForGit, as an option.

@dscho
Copy link
Member Author

dscho commented Feb 12, 2019

As VFSForGit overrides the version of Git that Visual Studio uses to the one that VFSForGit installs, this will cause issue with VFSForGit and Visual Studio 2017 installed on the same machine.

True. But VFS for Git has this option deprecated for ages now, too... So all that would be needed is to make sure that Team Explorer does not use the deprecated option, but instead the one sanctioned by the core Git reviewers. @clboles ... this is in your court now...

@dscho
Copy link
Member Author

dscho commented Feb 12, 2019

For lurkers: We will have to wait with merging this PR for a while, until we can be sure that no users of those deprecated options are left.

@git-for-windows-ci git-for-windows-ci changed the base branch from master to main June 17, 2020 18:11
@dscho
Copy link
Member Author

dscho commented May 16, 2022

This option is still used by the version of Git that ships with Visual Studio 2017.

It's time, I think.

@dscho dscho force-pushed the drop-deprecated-status-option branch from 87b5a52 to 19107ef Compare May 16, 2022 08:51
dscho added 2 commits May 16, 2022 12:27
We said we would remove --show-ignored-directory in v2.16.0 or v2.17.0,
but v2.36.1 is now the current version and we still have not removed it.
Let's do it now.

We had to wait for a while because some users were still on Visual
Studio 2017 (which used that option until a certain stage). But then,
users who upgrade Git will also upgrade Visual Studio, or so one would
hope, therefore they should not be affected by this change.

Signed-off-by: Johannes Schindelin <[email protected]>
We said we would remove --show-ignored-directory in v2.16.0 or v2.17.0,
but v2.36.1 is now the current version and we still have not removed it.
Let's do it now.

We had to wait for a while because some users were still on Visual
Studio 2017 (which used that option until a certain stage). But then,
users who upgrade Git will also upgrade Visual Studio, or so one would
hope, therefore they should not be affected by this change.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the drop-deprecated-status-option branch from 19107ef to 18ab665 Compare May 16, 2022 10:30
@dscho
Copy link
Member Author

dscho commented May 16, 2022

I will merge this soon, mainly as an excuse to get a new snapshot. For the same reason, I will hold off from merging #2068 for a bit, in case we need another snapshot soon.

@dscho dscho merged commit c6b0c5f into git-for-windows:main May 16, 2022
@dscho dscho deleted the drop-deprecated-status-option branch May 16, 2022 11:25
@dscho dscho added this to the Next release milestone May 16, 2022
dscho added a commit to git-for-windows/build-extra that referenced this pull request May 16, 2022
The Git for Windows-only `--show-ignored-directory` option of
`git status`, which was deprecated a long time ago, [was finally
removed](git-for-windows/git#2067).

Signed-off-by: Johannes Schindelin <[email protected]>
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

Successfully merging this pull request may close these issues.

4 participants