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

Add backers and sponsors to homepage #6537

Merged
merged 1 commit into from
Jun 28, 2018

Conversation

rickhanlonii
Copy link
Member

Summary

This PR adds the Open Collective backers and sponsors to the home page. This is updated per deploy by running a fetchSupporters.js script which creates a backers.json file used during the deploy process (i.e the list is static instead of dynamic)

Screen


Note: FB Open Source included twice just for an example

@cpojer
Copy link
Member

cpojer commented Jun 28, 2018

…aaaand we are back :D

@rickhanlonii rickhanlonii deleted the rh-contributors branch June 28, 2018 14:49
rickhanlonii added a commit that referenced this pull request Jun 28, 2018
## Summary

<!-- Explain the **motivation** for making this change. What existing problem does the pull request solve? -->
Adds the open collective blog post 

## Screen
![localhost_3000_blog_2018_06_27_supporting-jest-open-source 1](https://user-images.githubusercontent.com/2440089/42046107-1e245af6-7acb-11e8-960f-f7b35c245d8f.png)

## TODOs

Before launch we should:

- [x] Move the domain over to jestjs.io
- [x] Add the backers/sponsors to the docs #6458
- [x] Add backers/sponsors to the homepage #6537
@SimenB
Copy link
Member

SimenB commented Jun 29, 2018

This seems to have broken node 6? Not sure how

@connectdotz
Copy link
Contributor

hi, right now PRs are failing the ci/circleci: test-node-6 during yarn install, for example here:

[4/4] Building fresh packages...
error /home/circleci/jest/node_modules/jest-website: Command failed.
Exit code: 1
Command: node fetchSuporters.js
Arguments: 
Directory: /home/circleci/jest/node_modules/jest-website
Output:
/home/circleci/jest/website/fetchSuporters.js:23
          );
          ^

SyntaxError: Unexpected token )
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:549:28)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)
    at Function.Module._load (module.js:445:3)
    at Module.runMain (module.js:611:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:160:9)

looks like @cpojer merged the change even with node-6 failure, does that mean that jest no longer support node-6? In that case should we just drop node-6 from ci test? or somebody from the jest-website can fix it? what is your recommendation for PRs now seeing this ci error? ignore? wait? or...?

@rickhanlonii
Copy link
Member Author

@connectdotz it doesn't break node-6 support since that script is only for the website deploy, but we should fix since we may break node 6 without realizing it

@rickhanlonii
Copy link
Member Author

Was fixed here: #6587

@connectdotz
Copy link
Contributor

@rickhanlonii awesome! thx.

@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants