Skip to content
This repository has been archived by the owner on Apr 18, 2022. It is now read-only.

[FEATURE MODIFICATION] Simplify anonymous access of TeamCity build logs. #1366

Closed
udoprog opened this issue Jan 19, 2019 · 6 comments
Closed
Assignees
Labels
diff: easy Achievable by a single junior developer with a bit of guidance. pri: normal An issue that is causing a noticeable impact, but can be worked around team: infrastructure

Comments

@udoprog
Copy link
Contributor

udoprog commented Jan 19, 2019

Feature

Simplify anonymous access of TeamCity build logs.

Change

A contributor should not have to click "Login as Guest" to see build logs.

Reason

It's unintuitive compared to how build statuses usually work when posted to Github. This otherwise risks becoming a common support request on Discord or Github for new contributors.

Impact

Make new contributors more independent to fix any issues after they have set up a pull request.

Additional Notes

@udoprog udoprog changed the title [FEATURE MODIFICATION] Brief Description Here [FEATURE MODIFICATION] Simplify anonymous access of TeamCity build logs. Jan 19, 2019
@LucioFranco
Copy link
Member

One additional thing is that we might want to post build errors as PR comments with instructions on how to debug it. This should allow contributors to have an easier path to contributing.

@fhaynes fhaynes added diff: easy Achievable by a single junior developer with a bit of guidance. pri: normal An issue that is causing a noticeable impact, but can be worked around status: ready and removed status: needs triage labels Jan 19, 2019
@fhaynes
Copy link
Member

fhaynes commented Jan 19, 2019

Can you post that link to the example @LucioFranco

@fhaynes
Copy link
Member

fhaynes commented Jan 19, 2019

Our fork of the plugin that does this is here: https://github.com/amethyst/commit-status-publisher

We'll need to tweak it, build it, and install it. It isn't terribly difficult, and the code looks nice.

@LucioFranco
Copy link
Member

@fhaynes here you go rust-lang/rust#57578 (comment)

@fhaynes
Copy link
Member

fhaynes commented Jan 19, 2019

OK, this shouldn't be too difficult. I'll try to do it next week.

@fhaynes
Copy link
Member

fhaynes commented Mar 22, 2019

This is no longer relevant.

@fhaynes fhaynes closed this as completed Mar 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
diff: easy Achievable by a single junior developer with a bit of guidance. pri: normal An issue that is causing a noticeable impact, but can be worked around team: infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants