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

fix file url for enterprise #18

Merged
merged 3 commits into from
Nov 18, 2018
Merged

fix file url for enterprise #18

merged 3 commits into from
Nov 18, 2018

Conversation

tarek360
Copy link
Owner

@tarek360 tarek360 commented Nov 18, 2018

fix file url for github enterprise..

@corombo
Copy link
Collaborator

corombo commented Nov 18, 2018

Koshry Report

Status JaCoCo Coverage Report Coverage
🔥 io/github/tarek360/koshry/url/GithubFileUrlGenerator 0%
Project Total Coverage 88%

@tarek360 tarek360 merged commit 46638d3 into develop Nov 18, 2018
tarek360 added a commit that referenced this pull request Nov 18, 2018
* introduce JacocoCoverageRule (#6)

* introduce JacocoCoverageRule

* add JacocoCoverageRuleTest

* support all nested objects

* improve test coverage

* add more tests

* make issue levels take text (#8)

* config bintray

* convert the Rule to abstract class (#7)

* convert the Rule to abstract class

* fix push.gradle

* auto release (#9)

* auto release

* add release task to circle ci

* add ci and pullRequest to Rule (#10)

* add ci and pullRequest to Rule

* fix test

* exclude some authors from protected files rule (#11)

* jacoco total coverage (#12)

* introduce Github-enterprise (#15)

* Unify dependencies versions by applying buildSrc (#14)

* Making unify dependencies versions feature by applying buildSrc mechanism and Gradle DSL then change all the build.gradle.kts files of all the modules we have to be use the new kotlin class that I made.
Since four classes are made :
1. Dependencies : that contains all the dependencies we will use.
2. MainApp : that contains the common things of the whole app.
3. MainClass : that contains the main classes of each module.
4. Projects : that contains the name of projects/modules we have tell now.
5. Versions : that contains the versions that we will use in Dependencies class.

* Modify Naming of buildSrc files
- Projects file renamed to Modules to be more meaningful.
- MainApp file renamed to Project to be more specific.

* fix file url for enterprise (#18)

* fix file url for enterprise

* fix file path by using filePath instead of class path

* add UTs

* update comment (#19)

* update comment

* fix parsing

* make use of another github token

* release 0.0.4
tarek360 added a commit that referenced this pull request Nov 21, 2018
* introduce JacocoCoverageRule (#6)

* introduce JacocoCoverageRule

* add JacocoCoverageRuleTest

* support all nested objects

* improve test coverage

* add more tests

* make issue levels take text (#8)

* config bintray

* convert the Rule to abstract class (#7)

* convert the Rule to abstract class

* fix push.gradle

* auto release (#9)

* auto release

* add release task to circle ci

* add ci and pullRequest to Rule (#10)

* add ci and pullRequest to Rule

* fix test

* exclude some authors from protected files rule (#11)

* jacoco total coverage (#12)

* introduce Github-enterprise (#15)

* Unify dependencies versions by applying buildSrc (#14)

* Making unify dependencies versions feature by applying buildSrc mechanism and Gradle DSL then change all the build.gradle.kts files of all the modules we have to be use the new kotlin class that I made.
Since four classes are made :
1. Dependencies : that contains all the dependencies we will use.
2. MainApp : that contains the common things of the whole app.
3. MainClass : that contains the main classes of each module.
4. Projects : that contains the name of projects/modules we have tell now.
5. Versions : that contains the versions that we will use in Dependencies class.

* Modify Naming of buildSrc files
- Projects file renamed to Modules to be more meaningful.
- MainApp file renamed to Project to be more specific.

* fix file url for enterprise (#18)

* fix file url for enterprise

* fix file path by using filePath instead of class path

* add UTs

* update comment (#19)

* update comment

* fix parsing

* make use of another github token

* release 0.0.4

* support teamcity (#21)
@tarek360 tarek360 added the bug Something isn't working label Feb 13, 2019
tarek360 added a commit that referenced this pull request Aug 4, 2019
* introduce JacocoCoverageRule (#6)

* introduce JacocoCoverageRule

* add JacocoCoverageRuleTest

* support all nested objects

* improve test coverage

* add more tests

* make issue levels take text (#8)

* config bintray

* convert the Rule to abstract class (#7)

* convert the Rule to abstract class

* fix push.gradle

* auto release (#9)

* auto release

* add release task to circle ci

* add ci and pullRequest to Rule (#10)

* add ci and pullRequest to Rule

* fix test

* exclude some authors from protected files rule (#11)

* jacoco total coverage (#12)

* introduce Github-enterprise (#15)

* Unify dependencies versions by applying buildSrc (#14)

* Making unify dependencies versions feature by applying buildSrc mechanism and Gradle DSL then change all the build.gradle.kts files of all the modules we have to be use the new kotlin class that I made.
Since four classes are made :
1. Dependencies : that contains all the dependencies we will use.
2. MainApp : that contains the common things of the whole app.
3. MainClass : that contains the main classes of each module.
4. Projects : that contains the name of projects/modules we have tell now.
5. Versions : that contains the versions that we will use in Dependencies class.

* Modify Naming of buildSrc files
- Projects file renamed to Modules to be more meaningful.
- MainApp file renamed to Project to be more specific.

* fix file url for enterprise (#18)

* fix file url for enterprise

* fix file path by using filePath instead of class path

* add UTs

* update comment (#19)

* update comment

* fix parsing

* make use of another github token

* release 0.0.4

* support teamcity (#21)

* Create README.md

* pull request expose more info (#23)

* add custom rule post link

* Update README.md

* introduce debuggable flag (#24)

* update version to 0.0.4.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants