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

Fixed a bug on checks file. #17

Closed
wants to merge 1 commit into from
Closed

Fixed a bug on checks file. #17

wants to merge 1 commit into from

Conversation

aanm
Copy link
Contributor

@aanm aanm commented Feb 24, 2015

Added .idea directory to list of ignored files
Fixed a bug that occured if go-version had a PATCH version number

Signed-off-by: André Martins martins@noironetworks.com

Added .idea directory to list of ignored files
Fixed a bug that occured if go-version had a PATCH version number

Signed-off-by: André Martins <martins@noironetworks.com>
@@ -6,6 +6,7 @@
# Folders
_obj
_test
.idea/
Copy link
Contributor

Choose a reason for hiding this comment

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

just for my understanding, how is this directory used? Is it IDE specific?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is the IntelliJ IDEA

@mapuri
Copy link
Contributor

mapuri commented Mar 17, 2015

@aanm

closing this PR for now, as right now there is a strict dependency between go version on host and vagrant-vm.

However, this dependency does impose an unnecessarily strict restriction on the development environment. I will open a separate PR to make changes to the test-environment instead which can remove this dependency while not affecting the test times.

Hope that is ok.

@mapuri mapuri closed this Mar 17, 2015
@mapuri mapuri deleted the Fixing-scripts branch June 9, 2015 00:37
dseevr pushed a commit to dseevr-dev/netplugin that referenced this pull request Oct 24, 2017
* Hostname added

* Hostname is set in RegisterService
dseevr pushed a commit to dseevr-dev/netplugin that referenced this pull request Nov 28, 2017
* infranw: fixed model backward compatibility

* fix typo error

* Updated comments - default value for nwType
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.

None yet

2 participants