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.

Sign up for GitHub

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

Jump to bottom

Add Github Actions and COVERALLS_FLAG_NAME support #296

Merged
merged 36 commits into from Oct 5, 2020
Merged

Add Github Actions and COVERALLS_FLAG_NAME support #296

merged 36 commits into from Oct 5, 2020

Conversation

twinh
Copy link
Contributor

@twinh twinh commented Sep 28, 2020

  1. Add Github Actions, base on Add support for Github Actions #275
  2. Add COVERALLS_FLAG_NAME support

A working example, which contains Github Actions, parallel build and flag name.
https://github.com/twinh/github-actions-test/runs/1174413615?check_suite_focus=true

And the running result.
https://coveralls.io/builds/33756500

Smolevich and others added 30 commits October 21, 2019 23:18
@Smolevich
Add support for Github Actions
367798f
@Smolevich
Change typo
452ce4e
@Smolevich
Add return for Github Actions
e605806
@Smolevich
Fix failed test
f2fbd9e
@Smolevich
Fix cs errors
ac9dbc1
@Smolevich
Merge remote-tracking branch 'upstream/2.2' into add-support-for-gith…
bb38e5d 
…ub-actions
@Smolevich
Merge 2.2 and add blank yaml for GIthub Actions
16fed6b
@Smolevich
Fix github actions
da77b94
@Smolevich
Fix ci
98107a0
@Smolevich
Update exception message
ff5ced5
@simPod
GA tweaks
b9c9d4c 
https://github.com/coverallsapp/github-action/blob/master/src/run.ts
https://github.com/nickmerwin/node-coveralls/blob/336123fe05e7f80d79fc8522b4ff21a60fffe77c/lib/fetchGitData.js
@Smolevich
Reformat method fillGithubActions
cde6a12
@Smolevich
Add condition for pull request
3cfbf08
@Smolevich
Install dev tools in workflow file
f188c9e
@Smolevich
Fix cs fixer errors
39ed46b
@Smolevich
Merge branch 'add-support-for-github-actions' into add-support-for-gi…
dadbc8b 
…thub-actions
@Smolevich
Merge pull request #1 from simPod/add-support-for-github-actions
26e4c81 
GA tweaks
@Smolevich
Fix test CiEnvVarsCollectorTest
34dc99b
@Smolevich
Change collector and test for him
7b9bea1
@Smolevich
Change test
6eef3ee
@Smolevich
Fix test for GA
42b8068
@Smolevich
Fix failed tests
895f08e
@twinh
Merge remote-tracking branch 'upstream/2.3' into feature/github-actions
db8add4
@twinh
build: added PHP 7.4
d73a2cb 
(cherry picked from commit 6148248)
@twinh
refactor: use GITHUB_RUN_ID as CI_JOB_ID, same as coverallsapp/github…
743818d 
…-action

(cherry picked from commit 1d30901)
@twinh
docs: add link about GITHUB_RUN_ID
905ff53
@twinh
ci: Replace extension-csv to extension
342cc47 
Input 'extension-csv' has been deprecated with message: The extension-csv property will not be supported after February 1, 2020. Use extensions instead.
@twinh
test: fix job ID test fail
3f5d1af
@twinh
feat: add COVERALLS_FLAG_NAME support
381ecd0
@twinh
docs: add Github Actions setup
48a4ead
@twinh twinh mentioned this pull request Sep 28, 2020
Closed
@szepeviktor szepeviktor mentioned this pull request Sep 28, 2020
Closed
@keradus keradus merged commit f266bd4 into php-coveralls:2.3 Oct 5, 2020
@keradus
Copy link
Member

keradus commented Oct 5, 2020

Big thanks @Smolevich , @simPod and @twinh !!! :shipit:

@twinh twinh deleted the feature/github-actions branch October 9, 2020 03:22
@szepeviktor szepeviktor mentioned this pull request Oct 26, 2020
Merged
7 tasks
reececomo pushed a commit to reececomo/php-coveralls that referenced this pull request Jan 15, 2021
@tspencer244 @sanmai
@szepeviktor @dmitry-ivanov @keradus @dshemin @yihyang @twinh @Smolevich @simPod @DannyvdSluijs @oanhnn @jimmycleuren @craue
Merge upstream php-coveralls/php-coveralls:master (#4)
e382fe1 
* Update .travis.yml to include PHP 7.4 (php-coveralls#283)

* Fix incorrect version in README (php-coveralls#286)

* Add Guzzle 7 support (php-coveralls#288)

* Add Guzzle 7 support

* Update composer.json

Co-authored-by: Dariusz Rumiński <dariusz.ruminski@gmail.com>

* DX: Configurator - reduce cyclomatic complexity (php-coveralls#291)

* CI: reduce amount of jobs (php-coveralls#292)

* Allow to specify endpoint in arguments (php-coveralls#290)

* Allow to specify endpoint in arguments

* use new helper

Co-authored-by: Dariusz Ruminski <dariusz.ruminski@gmail.com>

* DX: Allow PHPUnit 7 (php-coveralls#294)

* Added COVERALLS_PARALLEL support and Configured CI_BUILD_NUMBER for Travis CI (php-coveralls#279)

* updated to detect COVERALLS_PARALLEL env var

* added CI_BUILD_NUMBER into Configuration

* updated README

* fixed tests

* Update RequirementsNotSatisfiedException.php

* fix CS

* fix checking of required fields + tests

* add missing class property

Co-authored-by: Dariusz Rumiński <dariusz.ruminski@gmail.com>

* Prepare v2.3.0

* Add support for PHP 8 + PHPUnit 9 (php-coveralls#295)

* Add support for PHP 8 + PHPUnit 9

* Add PHPUnit 4.8 workarounds

* Do --ignore-platform-reqs on PHP 8

* Skip failing test due to missing get_debug_type()

* Add insecure option (php-coveralls#289)

* Add Github Actions and COVERALLS_FLAG_NAME support (php-coveralls#296)

Co-authored-by: Smolevich <smolevich90@gmail.com>
Co-authored-by: Simon Podlipsky <simon@podlipsky.net>

* DX: .gitignore cache PHPUnit (php-coveralls#297)

* Prepare v2.4.0

* Fix support of branch name with hyphen and "(no branch)" (php-coveralls#298)

* Fix no-branch on CI error

* fix support of dash in branch names

* little tests cleanup

* Prepare v2.4.1

* Correct spelling error in Github (php-coveralls#299)

* Add COVERALLS_REPO_TOKEN to error message when run on Github Action (php-coveralls#302)

* Update README.md (php-coveralls#303)

Version 2.4.0 does not have a phar file, change to v2.4.2

* Add file missing for PHP 8 (php-coveralls#308)

fixes php-coveralls#307

* Prepare v2.4.3

* Update CiEnvVarsCollector.php

Re-arrange methods to avoid merge conflicts with upstream

* Update JsonFileTest.php

Remove duplicate method `shouldSetParallel()`

* Update JsonFile.php

Remove duplicate property `$parallel`

* Update JsonFile.php

Remove duplicate method `setParallel()`

* Update CiEnvVarsCollector.php

Fix code style

* Update JsonFileTest.php

Fix broken test

Co-authored-by: Alexey Kopytko <alexey@kopytko.com>
Co-authored-by: Viktor Szépe <viktor@szepe.net>
Co-authored-by: Dmitry Ivanov <dmitry.g.ivanov@gmail.com>
Co-authored-by: Dariusz Rumiński <dariusz.ruminski@gmail.com>
Co-authored-by: Dmitry Shemin <shemindmitry@gmail.com>
Co-authored-by: Yih Yang <yyngo788n88@hotmail.com>
Co-authored-by: Twin Huang <twinhuang@qq.com>
Co-authored-by: Smolevich <smolevich90@gmail.com>
Co-authored-by: Simon Podlipsky <simon@podlipsky.net>
Co-authored-by: Danny van der Sluijs <danny.vandersluijs@icloud.com>
Co-authored-by: Oanh Nguyen <oanhnn.bk@gmail.com>
Co-authored-by: Jimmy Cleuren <jimmy.cleuren@gmail.com>
Co-authored-by: Christian Raue <christian.raue@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanmai sanmai sanmai left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@twinh @keradus @sanmai @simPod @Smolevich