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

T: add test that intentions have descriptions and templates #6042

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented Aug 31, 2020

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

@Kobzol Kobzol added the internal Pull requests about internal improvements/fixes that don't affect users directly label Aug 31, 2020
@Kobzol Kobzol requested a review from Undin August 31, 2020 20:47
@Undin Undin self-assigned this Sep 1, 2020
bors bot added a commit that referenced this pull request Sep 2, 2020
6050: INT: add missing description and plugin registration for CreateFunctionIntention r=Undin a=Kobzol

Unblocks #6042

Co-authored-by: Jakub Beránek <jakub.beranek@vsb.cz>
bors bot added a commit that referenced this pull request Sep 2, 2020
6050: INT: add missing description and plugin registration for CreateFunctionIntention r=Undin a=Kobzol

Unblocks #6042

Co-authored-by: Jakub Beránek <jakub.beranek@vsb.cz>
bors bot added a commit that referenced this pull request Sep 2, 2020
6050: INT: add missing description and plugin registration for CreateFunctionIntention r=Undin a=Kobzol

Unblocks #6042

Co-authored-by: Jakub Beránek <jakub.beranek@vsb.cz>
Copy link
Member

@Undin Undin left a comment

Choose a reason for hiding this comment

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

Great!
Let's rebase onto master to fix test for CreateFunctionIntention intention

Copy link
Member

@Undin Undin left a comment

Choose a reason for hiding this comment

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

Thanks!
bors r+

bors bot added a commit that referenced this pull request Sep 3, 2020
6042: T: add test that intentions have descriptions and templates r=Undin a=Kobzol

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

Build failed:

  • check

@Undin
Copy link
Member

Undin commented Sep 3, 2020

bors retry

bors bot added a commit that referenced this pull request Sep 3, 2020
6042: T: add test that intentions have descriptions and templates r=Undin a=Kobzol

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

Build failed:

  • check

@Undin
Copy link
Member

Undin commented Sep 3, 2020

bors r+

bors bot added a commit that referenced this pull request Sep 3, 2020
6042: T: add test that intentions have descriptions and templates r=Undin a=Kobzol

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

Build failed:

  • check

@Kobzol
Copy link
Member Author

Kobzol commented Sep 3, 2020

bors retry

bors bot added a commit that referenced this pull request Sep 3, 2020
6042: T: add test that intentions have descriptions and templates r=Undin a=Kobzol

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

This PR was included in a batch that successfully built, but then failed to merge into master (it was a non-fast-forward update). It will be automatically retried.

bors bot added a commit that referenced this pull request Sep 3, 2020
6042: T: add test that intentions have descriptions and templates r=Undin a=Kobzol

This PR adds a test to all intentions that they have proper description and templates (inspired by a similar test in inspection test base class).

Blocked on #6050

Co-authored-by: Jakub Beránek <berykubik@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

Build failed:

  • check

@Kobzol
Copy link
Member Author

Kobzol commented Sep 3, 2020

@Undin is it possible to see why the merge failed? I can't see it on GitHub nor in BORS.

@Undin
Copy link
Member

Undin commented Sep 3, 2020

@Kobzol Yep.
image
image

@Undin
Copy link
Member

Undin commented Sep 3, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 3, 2020

Build succeeded:

@bors bors bot merged commit edc20fa into intellij-rust:master Sep 3, 2020
@Kobzol
Copy link
Member Author

Kobzol commented Sep 3, 2020

Oh, I knew that, but somehow I missed the red cross marks 🤦‍♂️ Thanks :)

@Kobzol Kobzol deleted the t-intention-description branch September 3, 2020 12:49
@Undin Undin added this to the v131 milestone Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Pull requests about internal improvements/fixes that don't affect users directly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants