Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for Teamcity running on custom context name #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

morincer
Copy link

No description provided.

@nskvortsov
Copy link
Collaborator

will it work for a newly downloaded TeamCity (e.g., when a user does not have TeamCity installed and the plugin downloads and unpacks it itself)?

@morincer
Copy link
Author

It shouldn't unless we allow the plugin to manipulate webapps. In fact everything my change does it just removing hard coding for the "ROOT" application name.

@nskvortsov
Copy link
Collaborator

In this case, please add some diagnostic message when a customized build is used in a non-applicable environment. Detecting missing application directory and failing early with meaningful error message (it should describe what has happened and what a user should do about it) will be good.

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

Successfully merging this pull request may close these issues.

2 participants