Skip to content

Update .gitignore for GoLand - #2

Merged
aravindhp merged 1 commit into
openshift:masterfrom
aravindhp:gitignore
Feb 24, 2020
Merged

aravindhp merged 1 commit into
openshift:masterfrom
aravindhp:gitignore

Conversation

@aravindhp

@aravindhp aravindhp commented Feb 20, 2020

Copy link
Copy Markdown
Contributor

Add GoLand config directory to .gitignore

@aravindhp

Copy link
Copy Markdown
Contributor Author

/cc @openshift/openshift-team-windows-containers

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 20, 2020
@openshift-ci-robot
openshift-ci-robot requested a review from a team February 20, 2020 19:00

@sebsoto sebsoto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 20, 2020

@sebsoto sebsoto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm cancel

Comment thread .gitignore Outdated

# Directory of IDE settings
*.idea/*
*.vscode/*

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From git ignore docs:

The pattern foo/ will match a directory foo and paths underneath it, but will not match a regular file or a symbolic link foo (this is consistent with the way how pathspec works in general in Git)

Do you need the trailing *?

Especially since:

The pattern "foo/*", matches "foo/test.json" (a regular file), "foo/bar" (a directory), but it does not match "foo/bar/hello.c" (a regular file), as the asterisk in the pattern does not match "bar/hello.c" which has a slash in it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied it over from the WMCB repo. We should fix it there too.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 20, 2020
@sebsoto

sebsoto commented Feb 20, 2020

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 20, 2020
Add GoLand config directory to .gitignore
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 21, 2020
@aravindhp aravindhp changed the title Update .gitignore Update .gitignore for GoLand Feb 22, 2020
@ravisantoshgudimetla

Copy link
Copy Markdown
Contributor

/approve

@suhanime

Copy link
Copy Markdown

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 24, 2020
@aravindhp
aravindhp merged commit 7dadac5 into openshift:master Feb 24, 2020
@aravindhp
aravindhp deleted the gitignore branch February 24, 2020 21:09
alinaryan added a commit to alinaryan/windows-machine-config-operator that referenced this pull request Apr 8, 2021
# This is the 1st commit message:

[ci] Add logs to network tests
The purpose of this commit is to add pod logs to
our networking tests to aid in seeing the network
communication errors.

# This is the commit message openshift#2:

[ci] adjust job name
wgahnagl pushed a commit to wgahnagl/windows-machine-config-operator that referenced this pull request Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants