Skip to content

Commit 10fc70c

Browse files
authored
Patched #54 (#55)
* chore: Simplified configurations * chore: Standardized files --------- Co-authored-by: ijlee2 <[email protected]>
1 parent 3e347f3 commit 10fc70c

File tree

8 files changed

+21
-62
lines changed

8 files changed

+21
-62
lines changed

.eslintrc.cjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ module.exports = {
3535
extends: ['plugin:@typescript-eslint/recommended-type-checked'],
3636
rules: {
3737
'@typescript-eslint/array-type': 'error',
38-
'@typescript-eslint/no-non-null-assertion': 'off',
3938
'@typescript-eslint/no-unsafe-argument': 'off',
4039
'@typescript-eslint/no-unsafe-assignment': 'off',
4140
},

.github/ISSUE_TEMPLATE/give-feedback.md

Lines changed: 0 additions & 23 deletions
This file was deleted.

.github/ISSUE_TEMPLATE/report-bug.md

Lines changed: 7 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -14,38 +14,23 @@ Before you make a new issue, please search for similar issues. It's possible tha
1414

1515
## Describe the bug 🐞
1616

17-
A clear and concise description of what the bug is.
17+
A clear and concise description of the bug.
1818

1919

2020
## Expected behavior 🤔
2121

22-
A clear and concise description of what you expected to happen.
22+
A clear and concise description of what you expected to see.
2323

2424

2525
## Minimal reproduction 🔬
2626

27-
Describe steps to reproduce the behavior:
27+
Describe steps to reproduce the issue.
2828

29-
1. Go to '...'
30-
2. Click on '...'
31-
3. Scroll down to '...'
32-
4. See error
29+
1. ...
30+
1. ...
31+
1. ...
3332

34-
If possible, please, share a link with a minimal reproduction.
35-
36-
<!--
37-
Create a minimal reproduction using one of the following:
38-
- Ember Twiddle: https://ember-twiddle.com/
39-
- Create a GitHub repository: https://guides.emberjs.com/release/getting-started/quick-start/
40-
-->
41-
42-
43-
## Environment 🌍
44-
45-
- Ember: -
46-
- Node.js/npm: -
47-
- OS: -
48-
- Browser: -
33+
If possible, please share a repo with the minimum files to reproduce the issue.
4934

5035

5136
## Additional context ➕

.prettierrc.cjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ module.exports = {
77
options: {
88
printWidth: 80,
99
singleQuote: true,
10-
trailingComma: 'all',
1110
},
1211
},
1312
],

README.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ In most cases, I recommend running the codemod without any arguments (i.e. allow
5353
By default, the package name decides where the addon package lives. Pass `--addon-location` to override the logic. This may be useful if you have a workspace with many addons.
5454

5555
```sh
56-
npx ember-codemod-v1-to-v2 --addon-location="packages/ui/buttons"
56+
npx ember-codemod-v1-to-v2 --addon-location packages/ui/buttons
5757
```
5858

5959
</details>
@@ -64,7 +64,7 @@ npx ember-codemod-v1-to-v2 --addon-location="packages/ui/buttons"
6464
Pass `--root` to run the codemod on a project somewhere else (i.e. not in the current directory).
6565

6666
```sh
67-
npx ember-codemod-v1-to-v2 --root=<path/to/your/project>
67+
npx ember-codemod-v1-to-v2 --root <path/to/your/project>
6868
```
6969

7070
</details>
@@ -75,7 +75,7 @@ npx ember-codemod-v1-to-v2 --root=<path/to/your/project>
7575
By default, the test-app package lives in the folder `test-app`. Pass `--test-app-location` to override the logic.
7676

7777
```sh
78-
npx ember-codemod-v1-to-v2 --test-app-location="docs-app"
78+
npx ember-codemod-v1-to-v2 --test-app-location docs-app
7979
```
8080

8181
</details>
@@ -86,7 +86,7 @@ npx ember-codemod-v1-to-v2 --test-app-location="docs-app"
8686
By default, the test-app package is named `test-app`. Pass `--test-app-name` to override the logic. This may be useful if you have a workspace with many addons.
8787

8888
```sh
89-
npx ember-codemod-v1-to-v2 --test-app-name="test-app-for-ui-buttons"
89+
npx ember-codemod-v1-to-v2 --test-app-name test-app-for-ui-buttons
9090
```
9191

9292
</details>
@@ -105,7 +105,7 @@ cd <path/to/cloned/repo>
105105
pnpm build
106106

107107
# Run codemod
108-
./dist/bin/ember-codemod-v1-to-v2.js --root=<path/to/your/project>
108+
./dist/bin/ember-codemod-v1-to-v2.js --root <path/to/your/project>
109109
```
110110

111111
You can also look at another codemod called [`ember-addon-migrator`](https://github.com/NullVoxPopuli/ember-addon-migrator).

codemod-test-fixture.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,15 @@
1515
#
1616
#---------
1717

18-
# Read the named arguments
19-
while getopts ":a:" flag
18+
# Get named arguments for the binary
19+
while getopts ":N:" flag
2020
do
2121
case $flag in
22-
a) ARGUMENTS=$OPTARG;;
22+
N) NAMED_ARGUMENTS=$OPTARG;;
2323
esac
2424
done
2525

26-
# Read the positional arguments
26+
# Get fixture name
2727
FIXTURE=${@:$OPTIND:1}
2828

2929
if [ ! $FIXTURE ]
@@ -39,6 +39,6 @@ fi
3939
rm -r "tests/fixtures/$FIXTURE/output"
4040
cp -r "tests/fixtures/$FIXTURE/input" "tests/fixtures/$FIXTURE/output"
4141

42-
./dist/bin/ember-codemod-v1-to-v2.js $ARGUMENTS --root="tests/fixtures/$FIXTURE/output"
42+
./dist/bin/ember-codemod-v1-to-v2.js $NAMED_ARGUMENTS --root="tests/fixtures/$FIXTURE/output"
4343

4444
echo "SUCCESS: Updated the output of $FIXTURE.\n"

codemod-test-fixtures.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
pnpm build
1717

1818
./codemod-test-fixture.sh \
19-
-a "--addon-location packages/ember-container-query --test-app-location demo-app --test-app-name demo-app-for-ember-container-query" \
19+
-N "--addon-location packages/ember-container-query --test-app-location demo-app --test-app-name demo-app-for-ember-container-query" \
2020
ember-container-query-customizations
2121

2222
./codemod-test-fixture.sh \
@@ -32,7 +32,7 @@ pnpm build
3232
ember-container-query-typescript
3333

3434
./codemod-test-fixture.sh \
35-
-a "--addon-location packages/new-v1-addon --test-app-location demo-app --test-app-name demo-app-for-new-v1-addon" \
35+
-N "--addon-location packages/new-v1-addon --test-app-location demo-app --test-app-name demo-app-for-new-v1-addon" \
3636
new-v1-addon-customizations
3737

3838
./codemod-test-fixture.sh \

package.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,16 +69,15 @@
6969
"typescript": "^5.1.6"
7070
},
7171
"engines": {
72-
"node": ">= 16"
72+
"node": "16.* || >= 18"
7373
},
7474
"changelog": {
7575
"labels": {
7676
"breaking": "Breaking Change",
7777
"bug": "Bug Fix",
7878
"enhance: code": "Enhancement",
7979
"enhance: dependency": "Internal",
80-
"enhance: documentation": "Documentation",
81-
"user feedback": "User Feedback"
80+
"enhance: documentation": "Documentation"
8281
}
8382
},
8483
"pnpm": {

0 commit comments

Comments
 (0)