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

data_source/github_repositories: add sort capabilities #234

Merged
merged 6 commits into from
Jun 8, 2019

Conversation

tracypholmes
Copy link
Contributor

Fixes #206 and added ability to sort

@ghost ghost added size/M Type: Documentation Improvements or additions to documentation labels May 31, 2019
@tracypholmes tracypholmes requested review from a team and removed request for bflad, ryndaniels, nywilken and mbfrahry June 3, 2019 16:13
Copy link
Contributor

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

Thanks for the updates @tracypholmes, I've left some minor comments inline that are mostly stylistic. Once those are addressed and tests pass this should be good to go 👍

github/data_source_github_repositories.go Outdated Show resolved Hide resolved
website/docs/d/repositories.html.markdown Outdated Show resolved Hide resolved
github/data_source_github_repositories.go Show resolved Hide resolved
github/data_source_github_repositories_test.go Outdated Show resolved Hide resolved
github/data_source_github_repositories_test.go Outdated Show resolved Hide resolved
@tracypholmes tracypholmes requested a review from katbyte June 7, 2019 20:44
…vider-github into f-github_repositories-sort-argument
--- PASS: TestAccGithubRepositoriesDataSource_basic (2.21s)
--- PASS: TestAccGithubRepositoriesDataSource_Sort (25.97s)
--- PASS: TestAccGithubRepositoriesDataSource_noMatch (1.60s)
@tracypholmes tracypholmes merged commit dffe362 into master Jun 8, 2019
@tracypholmes tracypholmes deleted the f-github_repositories-sort-argument branch June 19, 2019 04:37
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
…#234)

* data_source/github_repositories: add sort capabilities

* updates docs

* Update website/docs/d/repositories.html.markdown

Co-Authored-By: kt <[email protected]>

* Update github/data_source_github_repositories_test.go

Co-Authored-By: kt <[email protected]>

* reverts '%q' back to '"%s"'
--- PASS: TestAccGithubRepositoriesDataSource_basic (2.21s)
--- PASS: TestAccGithubRepositoriesDataSource_Sort (25.97s)
--- PASS: TestAccGithubRepositoriesDataSource_noMatch (1.60s)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

There is no option to sort repositories in github_repositories data source
2 participants