Skip to content

Commit

Permalink
Split profiles into track folders (#1452)
Browse files Browse the repository at this point in the history
* remove pp2002, closes #1431

* remove profile-amended

* fix test

* remove unused test file

* split profiles into track folders

* make track/profiles work

* Revert "make track/profiles work"

This reverts commit f52af89.

* Revert "Revert "make track/profiles work""

This reverts commit 14e624b.

* add all profiles under track

* align profiles

* give error message when doing unecessary rule remove

* Update lib/profiles/profileUtil.js

Co-authored-by: Denis Ah-Kang <[email protected]>

* Update lib/rules/metadata/profile.js

Co-authored-by: Denis Ah-Kang <[email protected]>

* Update lib/rules/metadata/profile.js

Co-authored-by: Denis Ah-Kang <[email protected]>

* Update lib/profiles/TR/Recommendation/FPWD.js

Co-authored-by: Denis Ah-Kang <[email protected]>

* remove usage rule from DRY, CRY, CRYD, fix lint

* fix test

Co-authored-by: Denis Ah-Kang <[email protected]>
  • Loading branch information
jennyliang220 and deniak authored Dec 8, 2021
1 parent 92ee16e commit e468eb5
Show file tree
Hide file tree
Showing 50 changed files with 487 additions and 376 deletions.
10 changes: 6 additions & 4 deletions lib/profiles/SUBM.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,10 @@

exports.name = 'Submission';

const base = require('./base');
const profileUtil = require('./profileUtil');

exports.rules = base.extendWithInserts({
'headers.logo': require('../rules/headers/subm-logo'),
});
exports.rules = profileUtil.insertAfter(
require('./base').rules,
'headers.logo',
require('../rules/headers/subm-logo')
);
12 changes: 7 additions & 5 deletions lib/profiles/SUBM/MEM-SUBM.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,13 @@ exports.config = {
submissionType: 'member',
};

const base = require('../base');
let rules = base.insertAfter(require('../SUBM').rules, 'headers.w3c-state', [
require('../../rules/headers/memsub-copyright'),
]);
rules = base.insertAfter(
const profileUtil = require('../profileUtil');
let rules = profileUtil.insertAfter(
require('../SUBM').rules,
'headers.w3c-state',
[require('../../rules/headers/memsub-copyright')]
);
rules = profileUtil.insertAfter(
rules,
'sotd.supersedable',
require('../../rules/sotd/submission')
Expand Down
31 changes: 16 additions & 15 deletions lib/profiles/TR.js
Original file line number Diff line number Diff line change
@@ -1,19 +1,20 @@
// base profile for all things TR
exports.name = 'TR';

const base = require('./base');
const profileUtil = require('./profileUtil');
const base = require('./base').rules;

exports.rules = base.extendWithInserts({
'headers.w3c-state': [
require('../rules/headers/github-repo'),
require('../rules/headers/copyright'),
],
'sotd.supersedable': [
require('../rules/sotd/stability'),
require('../rules/sotd/publish'),
require('../rules/sotd/draft-stability'),
require('../rules/sotd/pp'),
require('../rules/sotd/charter'),
require('../rules/sotd/process-document'),
],
});
let rules = profileUtil.insertAfter(base, 'headers.w3c-state', [
require('../rules/headers/github-repo'),
require('../rules/headers/copyright'),
]);

rules = profileUtil.insertAfter(rules, 'sotd.supersedable', [
require('../rules/sotd/stability'),
require('../rules/sotd/publish'),
require('../rules/sotd/pp'),
require('../rules/sotd/charter'),
require('../rules/sotd/process-document'),
]);

exports.rules = rules;
11 changes: 0 additions & 11 deletions lib/profiles/TR/CR-Echidna.js

This file was deleted.

21 changes: 0 additions & 21 deletions lib/profiles/TR/CR.js

This file was deleted.

11 changes: 0 additions & 11 deletions lib/profiles/TR/CRD-Echidna.js

This file was deleted.

20 changes: 0 additions & 20 deletions lib/profiles/TR/CRD.js

This file was deleted.

17 changes: 0 additions & 17 deletions lib/profiles/TR/CRY.js

This file was deleted.

18 changes: 0 additions & 18 deletions lib/profiles/TR/CRYD.js

This file was deleted.

12 changes: 0 additions & 12 deletions lib/profiles/TR/DISC.js

This file was deleted.

10 changes: 0 additions & 10 deletions lib/profiles/TR/DNOTE-Echidna.js

This file was deleted.

16 changes: 0 additions & 16 deletions lib/profiles/TR/DNOTE.js

This file was deleted.

16 changes: 0 additions & 16 deletions lib/profiles/TR/DRY.js

This file was deleted.

9 changes: 0 additions & 9 deletions lib/profiles/TR/FPWD.js

This file was deleted.

9 changes: 0 additions & 9 deletions lib/profiles/TR/NOTE-Echidna.js

This file was deleted.

18 changes: 0 additions & 18 deletions lib/profiles/TR/NOTE.js

This file was deleted.

13 changes: 13 additions & 0 deletions lib/profiles/TR/Note/DNOTE-Echidna.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// TODO: merge all Echidna files.
exports.name = 'DNOTE-Echidna';
const base = require('./DNOTE');

exports.config = base.config;

// customize rules
const profileUtil = require('../../profileUtil');
const rules = profileUtil.insertAfter(base.rules, 'sotd.process-document', [
require('../../../rules/echidna/todays-date'),
]);

exports.rules = rules;
18 changes: 18 additions & 0 deletions lib/profiles/TR/Note/DNOTE.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
exports.name = 'DNOTE';
const base = require('./note-base');

// customize config
const config = {
status: 'DNOTE',
longStatus: 'Group Draft Note',
styleSheet: 'W3C-DNOTE',
};
exports.config = { ...base.config, ...config };

// customize rules
const profileUtil = require('../../profileUtil');
const rules = profileUtil.insertAfter(base.rules, 'sotd.pp', [
require('../../../rules/sotd/draft-stability'),
]);

exports.rules = rules;
9 changes: 9 additions & 0 deletions lib/profiles/TR/Note/NOTE-Echidna.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
exports.name = 'NOTE-Echidna';
const base = require('./NOTE');

exports.config = base.config;

const profileUtil = require('../../profileUtil');
exports.rules = profileUtil.insertAfter(base.rules, 'sotd.process-document', [
require('../../../rules/echidna/todays-date'),
]);
12 changes: 12 additions & 0 deletions lib/profiles/TR/Note/NOTE.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
exports.name = 'NOTE';
const base = require('./note-base');

// customize config
const config = {
status: 'NOTE',
longStatus: 'Group Note',
styleSheet: 'W3C-NOTE',
};
exports.config = { ...base.config, ...config };

exports.rules = base.rules;
12 changes: 12 additions & 0 deletions lib/profiles/TR/Note/STMT.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
exports.name = 'STMT';
const base = require('./note-base');

// customize config
const config = {
status: 'STMT',
longStatus: 'Statement',
styleSheet: 'W3C-STMT',
};
exports.config = { ...base.config, ...config };

exports.rules = base.rules;
13 changes: 13 additions & 0 deletions lib/profiles/TR/Note/note-base.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
exports.config = {
track: 'Note',
};

// customize rules
const base = require('../../TR');
const profileUtil = require('../../profileUtil');

const rules = profileUtil.insertAfter(base.rules, 'sotd.pp', [
require('../../../rules/sotd/deliverer-note'),
]);

exports.rules = rules;
20 changes: 0 additions & 20 deletions lib/profiles/TR/PR.js

This file was deleted.

24 changes: 0 additions & 24 deletions lib/profiles/TR/REC-RSCND.js

This file was deleted.

Loading

0 comments on commit e468eb5

Please sign in to comment.