Skip to content

Commit

Permalink
Merge pull request #1116 from ember-learn/tooling-team
Browse files Browse the repository at this point in the history
merge cli, typescript, and embroider teams
  • Loading branch information
MinThaMie authored Jul 8, 2024
2 parents 02a85e6 + 160c842 commit 57d8dc4
Show file tree
Hide file tree
Showing 14 changed files with 32 additions and 44 deletions.
12 changes: 2 additions & 10 deletions app/controllers/teams.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,8 @@ export default class TeamsController extends Controller {
return this.sortedTeamMembers.filter(inTeam('alumni'));
}

get coreCLITeamMembers() {
return this.sortedTeamMembers.filter(inTeam('cli'));
}

get embroiderTeamMembers() {
return this.sortedTeamMembers.filter(inTeam('embroider'));
get coreToolingTeamMembers() {
return this.sortedTeamMembers.filter(inTeam('tooling'));
}

get coreTeamMembers() {
Expand All @@ -38,10 +34,6 @@ export default class TeamsController extends Controller {
return this.sortedTeamMembers.filter(inTeam('learning'));
}

get typescriptTeamMembers() {
return this.sortedTeamMembers.filter(inTeam('typescript'));
}

get steeringCommitteeMembers() {
return this.sortedTeamMembers.filter(inTeam('steering'));
}
Expand Down
22 changes: 3 additions & 19 deletions app/templates/teams.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -27,17 +27,9 @@

<section class="mb-6">
<Teams::Team
@description="The Ember CLI core team is responsible for maintaining ember-cli, the command line interface for managing and packaging Ember.js applications and addons. The team also maintains many of the addons in the default blueprint as well as fastboot."
@members={{this.coreCLITeamMembers}}
@name="The Ember CLI Core Team"
/>
</section>

<section class="mb-6">
<Teams::Team
@description="The Embroider Core Team is responsible for the Embroider build system."
@members={{this.embroiderTeamMembers}}
@name="The Embroider Core Team"
@description="The Ember Tooling core team is responsible for maintaining ember-cli, Embroider, Glint, Ember's TypeScript integration, and anything related to the Ember build system. The team also maintains many of the addons in the default blueprint as well as fastboot."
@members={{this.coreToolingTeamMembers}}
@name="The Ember Tooling Core Team"
/>
</section>

Expand All @@ -57,14 +49,6 @@
/>
</section>

<section class="mb-6">
<Teams::Team
@description="The Ember TypeScript core team is responsible for Ember’s TypeScript integration, including build tooling, linting, and collaborating with the Framework and Learning teams on API design and teaching materials."
@members={{this.typescriptTeamMembers}}
@name="The Ember TypeScript Core Team"
/>
</section>

<section>
<Teams::Team
@description="Serving as a member of the Core Team(s) of an open source project like Ember is a huge amount of work. These are the emeritus members of the Ember Core Teams, to whom we will always be grateful."
Expand Down
2 changes: 1 addition & 1 deletion data/team-member/aaron-chambers.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/grandazz'
image: achambers.jpg
added: 2023-05-25T13:01:14.573Z
teams:
- embroider
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/chris-manson.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ image: cmanson.jpg
added: 2018-10-20T09:01:00.000Z
teams:
- learning
- embroider
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/edward-faulkner.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,5 @@ added: 2015-04-08T09:01:00.000Z
teams:
- corejs
- steering
- embroider
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/james-davis.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/jamscdavis'
image: jdavis.jpg
added: 2022-03-09T18:25:00.000Z
teams:
- typescript
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/katie-gengler.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,6 @@ image: kgengler.jpg
added: 2016-04-14T09:03:00.000Z
teams:
- corejs
- cli
- tooling
- steering
---
2 changes: 1 addition & 1 deletion data/team-member/kelly-selden.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/kellyselden'
image: kellyselden.jpg
added: 2016-04-16T09:00:00.000Z
teams:
- cli
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/krystan-huffmenne.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ twitter: 'https://twitter.com/hm_krystan'
image: khuffmenne.jpg
added: 2023-05-19T00:00:00.000Z
teams:
- typescript
- tooling
- data
---
2 changes: 1 addition & 1 deletion data/team-member/peter-meehan.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/22a'
image: pmeehan.jpg
added: 2023-08-11T13:00:00Z
teams:
- embroider
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/peter-wagenet.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ image: pwagenet.jpg
added: 2013-04-02T09:02:00.000Z
teams:
- corejs
- typescript
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/preston-sego.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/nullvoxpopuli'
image: psego.jpeg
added: 2023-05-25T14:22:20.616Z
teams:
- embroider
- tooling
---
2 changes: 1 addition & 1 deletion data/team-member/simon-ihmig.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ twitter: 'https://twitter.com/simonihmig'
image: sihmig.jpeg
added: 2023-07-01T16:50:00Z
teams:
- embroider
- tooling
---
20 changes: 16 additions & 4 deletions tests/unit/controllers/teams-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,25 @@ module('Unit | Controller | teams', function (hooks) {
]);
});

test('We sort CLI team members by when they were first added', function (assert) {
const { coreCLITeamMembers } = this.controller;
test('We sort Tooling team members by when they were first added', function (assert) {
const { coreToolingTeamMembers } = this.controller;

// Create an intermediate data structure for assertion
const output = coreCLITeamMembers.map((teamMember) => teamMember.id);
const output = coreToolingTeamMembers.map((teamMember) => teamMember.id);

assert.deepEqual(output, ['katie-gengler', 'kelly-selden']);
assert.deepEqual(output, [
'peter-wagenet',
'edward-faulkner',
'katie-gengler',
'kelly-selden',
'chris-manson',
'james-davis',
'krystan-huffmenne',
'aaron-chambers',
'preston-sego',
'simon-ihmig',
'peter-meehan',
]);
});

test('We sort Data team members by when they were first added', function (assert) {
Expand Down

0 comments on commit 57d8dc4

Please sign in to comment.