Skip to content
This repository was archived by the owner on Mar 10, 2021. It is now read-only.

Conversation

@sergirubio
Copy link

No description provided.

Copy link
Collaborator

@piogor piogor left a comment

Choose a reason for hiding this comment

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

Thanks, @sergirubio, there is a need for a small correction to avoid the only warning ;-).

Comment on lines +70 to +72
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).
Copy link
Collaborator

Choose a reason for hiding this comment

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

It seem the identation has to be corrected (/home/travis/build/tango-controls/tango-doc/source/development/advanced/known-issues.rst:71: WARNING: Bullet list ends without a blank line; unexpected unindent.)

Suggested change
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).

Copy link
Author

Choose a reason for hiding this comment

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

hi piotr, I agree with the change
thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

@sergirubio, I think it is you who should apply the suggestion as I do not have write permission to ALBA repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants