Skip to content

Conversation

@Marcono1234
Copy link
Contributor

#3546 changed the query but did not adjust the query link.

Additionally the old query cannot be re-run because some of the projects it targeted (gradle/gradle and eclipse-cdt/cdt) cannot be queried currently.
The new query now queries all available demo projects of the query console instead.
Note that this affects some of the other query links as well, should the be updated too for consistency? Otherwise one part of the tutorial shows result for projects which the next part does not show.

github#3546 changed the query but did not adjust the query link.
Additionally the old query could not be re-run because some of the projects it
targeted (gradle/gradle and eclipse-cdt/cdt) cannot be queried currently.
It now queries all available demo projects of the query console instead.
@shati-patel
Copy link
Contributor

Thank you for your contribution @Marcono1234!

Note that this affects some of the other query links as well, should the be updated too for consistency? Otherwise one part of the tutorial shows result for projects which the next part does not show.

You're right, it makes sense to update the other links too. It would be great if you could include the updates in this PR too! (Though we can also do it in a follow-up PR, if you prefer.)

Thank you again for your help 😃

@shati-patel shati-patel removed the request for review from jf205 June 30, 2020 08:03
@aschackmull aschackmull merged commit 50fee5c into github:master Jul 2, 2020
@Marcono1234 Marcono1234 deleted the patch-1 branch July 5, 2020 16:47
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.

3 participants