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

Extend API #194

Closed
wants to merge 389 commits into from
Closed

Extend API #194

wants to merge 389 commits into from

Conversation

4np
Copy link
Contributor

@4np 4np commented Dec 4, 2024

Context

This PR:

  • extends the PullRequest API to:
    • fetch code review comments (see doc)
    • create a new regular code review comment (an alias for creating issue comment, as documented by GitHub, see doc)
    • create a new code review comment by commit ID, path and line number (see doc)
  • adds a new Comment structure, scoped onto PullRequest as it differs from the issue comment.
  • scopes the existing Comment structure onto Issue
  • adds unit tests for the new logic

muhdmirzamz and others added 30 commits July 25, 2017 16:22
Updated documentation in README to align with Swift 3
Add support for Swift Package Manager
* Remove not used Color extension

* Duplicate Models to compile also with Linux

* Added allTests property to make the tests executable also on Linux

* Updated .travis.yml to use also Linux

* Renamed travis scripts

* Use the correct Script folder name

* Updated permissions for travis scripts

* Use swift version 4.2 on Linux

* Added LinuxMain.swift

* Try to make swift find LinuxMain file

* Update LinuxMain permissions

* Reverse 21892d6

* Updated macos install script

* install bundler on macos

* Use SPM default test location to make it find LinuxMain.swift

* Fixed defaultTestSuite compilation issue for macOS and tvOS

* Set correct path to the Tests info.plist

* Don't use Bundle(for:) on Linux

* Not use the Bundle at all on Linux

* Make tests compile

* Remove all the other Bundle(for:) calls on Linux

* Fix: Use resourceName

* Find files also on Fixtures folder

* Remove objc compatibility

* Remove spaces from User.swift file
cwagdev and others added 27 commits February 7, 2023 09:01
Having this causes the wrong URL to be composed for enterprise hosted environment
* Add Post Review endpoint

* Add test case

* fix build error

* run `swiftformat`
Bumps [activesupport](https://github.com/rails/rails) from 6.1.7.2 to 6.1.7.3.
- [Release notes](https://github.com/rails/rails/releases)
- [Changelog](https://github.com/rails/rails/blob/v7.0.4.3/activesupport/CHANGELOG.md)
- [Commits](rails/rails@v6.1.7.2...v6.1.7.3)

---
updated-dependencies:
- dependency-name: activesupport
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Add swiftformat plugin for Package builds

* Add Swiftformat vscode plugin

* Fix devcontainer customizations

* Use latest version
* Add FixtureGenerator

* Add list option to gists

* Sort keys

* Add sorted keys command

* Add generate all fixtures command

* Add Issue command

* Add repository command

* Add Followers

* Add label commands

* Add PullRequest command

* Add release command

* Add Review command

* Add star command

* Add status command

* Add user command

* Run swiftformat

* Add some info in README

* Remove platforms

* Rename to OctokitCLI

* Compile CLI

* Use Working Directory

* Fix compiling on Linux

* Change abstract
* Adding create pull request support

* Updates from review

* Add async version

* Adding tests

* Support older swift versions

* Formatting tweaks to satisfy linter.
* add init for User

* add init for Plan

* add init for Status

* add init for Review

* add init for Repository

* add init for File

* add init for Gist

* add init for Issue

* add init for Label

* add init for Milestone

* add init for PullRequest

* add init for Reactions

* add init for Release

* format Plan

* execute SwiftFormat
* add endpoint for `List pull requests files`

* replace with enum

* fix accessibility

* add test
* Use package for linting instead of clone

* Update swiftformat
* Add pagination to PullRequests

* Adding perPage to PullRequests

---------

Co-authored-by: Seth Faxon <sfaxon@adobe.com>
* Run CI on main

* Remove more master things
Co-authored-by: Seth Faxon <sfaxon@adobe.com>
* Add search code

* Fix query param key

* Fix search query key

* Fix linting
* dateDecodingStrategy added as described in #184 (comment)

* dateDecodingStrategy added as described in Issue #184.
* Add milestone api

* Add async methods

* Use string instead of date

* Update page properties

* Update Milestone.swift

* Format code using swiftformat

* Use old if lets

* Update Milestone.swift

* Update MilestoneTests.swift

* Update MilestoneTests.swift

* Update state property docs

* Update Milestone.swift
* `Review.submittedAt` may be nil when just only created

* add `Delete a pending review for a pull request` endpoint

* add `Submit a review for a pull request` endpoint

* add tests

* replace `load` with `post`

* run swiftformat

* bump RequestKit to 3.3.0

* Resolve `FIXME`s

* fix RequestKit version in xcodeproj
* Add API implementation for repository contents

* Add unit tests for Repository Contents
* Add support for async await on all notification endpoints

* Updated documentation and added mark notification thread as read/done methods

* removed id from Subject, as it does not have one

* Formatting + added constructors

* Added async tests

* lint

* lint

* more linting
… async await (#192)

* Update Configuration.swift

Added handleOpenURL and Authorize support async await

* Update Configuration.swift

lint
* add some more fields to repo, and structure for organization

* add API to read a user's starred repositories

* add test for repo updates

* test org decoding; remove some invalid properties

* remove duplicate methods to fetch starred repos

* add Organization to all necessary targets

* use local fork of RequestKit

* fix tests

* fix spm tool version

* put back RequestKit dep

* fix spm tool version again

* fix lint issues

---------

Co-authored-by: Andrew McKnight <git@mcknight.rocks>
…egular comments as well as comments to specific commits / line numbers)
@4np
Copy link
Contributor Author

4np commented Dec 4, 2024

It looks like your CI isn't working as expected. This compiles and tests fine on Xcode 15.4 and Xcode 16.1.

@4np 4np force-pushed the scratch/extend-API branch from 799287f to 61e97e6 Compare December 4, 2024 16:15
@4np 4np closed this by deleting the head repository Dec 4, 2024
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.