Skip to content

Commit b1369ce

Browse files
committed
Merge latest
1 parent ca49741 commit b1369ce

File tree

788 files changed

+103387
-68575
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

788 files changed

+103387
-68575
lines changed

.codeclimate.yml

+3-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ plugins:
44
enabled: true
55
config:
66
languages:
7-
- javascript
7+
- javascript
88
exclude_patterns:
99
- 'cypress/{integration,support}/**/*.js'
1010
# The line below should work but doesn't it removes all js and jsx quality checking
@@ -17,3 +17,5 @@ exclude_patterns:
1717
- 'scripts/lighthouseBudget.test.js'
1818
- 'scripts/lighthouseBudget.js'
1919
- 'scripts/dependencyCheck.js'
20+
- 'scripts/simorghPages.js'
21+
- 'scripts/simorghLaunchDates.js'

.eslintrc.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ module.exports = {
3939
],
4040
},
4141
],
42-
"import/extensions": [1, { "json": "ignorePackages" }]
42+
'import/extensions': [1, { json: 'ignorePackages' }],
4343
},
4444
settings: {
4545
'import/resolver': {

.github/CODE_OF_CONDUCT.md

+12-10
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
### Documentation index
2+
23
Please familiarise yourself with our:
4+
35
- [Code of conduct](https://github.com/bbc/simorgh/blob/latest/.github/CODE_OF_CONDUCT.md) (you are here)
46
- [Code Standards](https://github.com/bbc/simorgh/blob/latest/docs/Code-Standards.md)
57
- [Contributing guidelines](https://github.com/bbc/simorgh/blob/latest/CONTRIBUTING.md)
@@ -18,19 +20,19 @@ In the interest of fostering an open and welcoming environment, we as contributo
1820

1921
Examples of behavior that contributes to creating a positive environment include:
2022

21-
* Using welcoming and inclusive language
22-
* Being respectful of differing viewpoints and experiences
23-
* Gracefully accepting constructive criticism
24-
* Focusing on what is best for the community
25-
* Showing empathy towards other community members
23+
- Using welcoming and inclusive language
24+
- Being respectful of differing viewpoints and experiences
25+
- Gracefully accepting constructive criticism
26+
- Focusing on what is best for the community
27+
- Showing empathy towards other community members
2628

2729
Examples of unacceptable behavior by participants include:
2830

29-
* The use of sexualized language or imagery and unwelcome sexual attention or advances
30-
* Trolling, insulting/derogatory comments, and personal or political attacks
31-
* Public or private harassment
32-
* Publishing others' private information, such as a physical or electronic address, without explicit permission
33-
* Other conduct which could reasonably be considered inappropriate in a professional setting
31+
- The use of sexualized language or imagery and unwelcome sexual attention or advances
32+
- Trolling, insulting/derogatory comments, and personal or political attacks
33+
- Public or private harassment
34+
- Publishing others' private information, such as a physical or electronic address, without explicit permission
35+
- Other conduct which could reasonably be considered inappropriate in a professional setting
3436

3537
## Our Responsibilities
3638

.github/PULL_REQUEST_TEMPLATE.md

+4-2
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
Resolves #NUMBER
22

3-
**Overall change:**
3+
**Overall change:**
44
_A very high-level summary of easily-reproducible changes that can be understood by non-devs._
55

66
**Code changes:**
7+
78
- _A bullet point list of key code changes that have been made._
89
- _When describing code changes, try to communicate **how** and **why** you implemented something a specific way, not just **what** has changed._
910

@@ -14,6 +15,7 @@ _A very high-level summary of easily-reproducible changes that can be understood
1415
- [ ] I have assigned this PR to the Simorgh project
1516

1617
**Testing:**
18+
1719
- [ ] Automated (jest and/or cypress) tests added (for new features) or updated (for existing features)
18-
- [ ] If necessary, I have run the local E2E non-smoke tests relevant to my changes (`CYPRESS_APP_ENV=local CYPRESS_SMOKE=false npm run test:e2e:interactive`)
20+
- [ ] If necessary, I have run the local E2E non-smoke tests relevant to my changes (`CYPRESS_APP_ENV=local CYPRESS_SMOKE=false npm run test:e2e:interactive`)
1921
- [ ] This PR requires manual testing

.github/workflows/simorgh_ci.yml

+12-13
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,22 @@ on: [push]
44

55
jobs:
66
build:
7-
87
runs-on: ubuntu-latest
98

109
strategy:
1110
matrix:
1211
node-version: [12.x]
1312

1413
steps:
15-
- uses: actions/checkout@v1
16-
- name: Use Node.js ${{ matrix.node-version }}
17-
uses: actions/setup-node@v1
18-
with:
19-
node-version: ${{ matrix.node-version }}
20-
- name: npm install, build, and test
21-
run: |
22-
npm ci
23-
npx apache2-license-checker
24-
npm test
25-
env:
26-
CI: true
14+
- uses: actions/checkout@v1
15+
- name: Use Node.js ${{ matrix.node-version }}
16+
uses: actions/setup-node@v1
17+
with:
18+
node-version: ${{ matrix.node-version }}
19+
- name: npm install, build, and test
20+
run: |
21+
npm ci
22+
npx apache2-license-checker
23+
npm test
24+
env:
25+
CI: true

.prettierrc

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
{
1+
{
22
"singleQuote": true,
33
"trailingComma": "all"
4-
}
4+
}

.storybook/.eslintrc

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
2-
"globals": {
2+
"globals": {
33
"cy": true,
44
"Cypress": true
5-
},
5+
}
66
}

.storybook/config.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import GlobalStyles from '@bbc/psammead-styles/global-styles';
77
const req = require.context('../src/app', true, /\.stories\.jsx$/);
88

99
function loadStories() {
10-
req.keys().forEach(filename => req(filename));
10+
req.keys().forEach((filename) => req(filename));
1111
}
1212

1313
const fontPathMap = [
@@ -24,13 +24,13 @@ const fontPathMap = [
2424
{ prefix: 'F_SHONAR_BANGLA', path: 'fonts/ShonarBangla/' },
2525
];
2626

27-
addDecorator(story => (
27+
addDecorator((story) => (
2828
/* eslint-disable react/jsx-filename-extension */
2929
<>
3030
<GlobalStyles
31-
fonts={Object.values(fontFaces).map(fontFace => {
31+
fonts={Object.values(fontFaces).map((fontFace) => {
3232
const fontMap =
33-
fontPathMap.find(map => fontFace.name.includes(map.prefix)) ||
33+
fontPathMap.find((map) => fontFace.name.includes(map.prefix)) ||
3434
fontPathMap[0];
3535
return fontFace(fontMap.path);
3636
})}

.storybook/helpers/ampDecorator/index.jsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import {
1313
Contains AMP required markup https://www.ampproject.org/docs/fundamentals/spec#required-markup
1414
*/
1515

16-
const AmpDecorator = storyFn => (
16+
const AmpDecorator = (storyFn) => (
1717
<div>
1818
<Helmet htmlAttributes={{ amp: '' }}>
1919
<link rel="canonical" href="http://foobar.com" />

0 commit comments

Comments
 (0)