Skip to content

Commit 27873ea

Browse files
authored
doc: update github templates (#314)
* doc: update github templates * doc: add pubspec section
1 parent e87328e commit 27873ea

File tree

7 files changed

+149
-10
lines changed

7 files changed

+149
-10
lines changed
Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
name: Bug Report
2+
description: File a bug report
3+
title: "[BUG]: "
4+
labels: ["bug"]
5+
body:
6+
- type: markdown
7+
attributes:
8+
value: |
9+
Thanks for taking the time to fill out this bug report!
10+
- type: checkboxes
11+
attributes:
12+
label: Is there an existing issue for this?
13+
description: Please search to see if an issue already exists for the bug you encountered.
14+
options:
15+
label: I have searched the existing issues
16+
required: true
17+
- type: input
18+
id: version
19+
attributes:
20+
label: Version
21+
description: What version of our software are you running? ($ fluttergen -v)
22+
placeholder: ex. 5.0.0
23+
validations:
24+
required: false
25+
- type: dropdown
26+
id: command-type
27+
attributes:
28+
label: Command type
29+
description: What version of our software are you running?
30+
options:
31+
- build_runner (Default)
32+
- Dart command
33+
- Homebrew
34+
validations:
35+
required: true
36+
- type: textarea
37+
id: what-happened
38+
attributes:
39+
label: What happened?
40+
description: Also tell us, what did you expect to happen?
41+
placeholder: Tell us what you see!
42+
value: "A bug happened!"
43+
validations:
44+
required: true
45+
- type: textarea
46+
id: pubspec
47+
attributes:
48+
label: Relevant a pubspec.yaml.
49+
description: Please copy and pates your pubspec.yaml.
50+
placeholder: ex. $ cat pubspec.yaml | pbcopy
51+
render: yaml
52+
- type: textarea
53+
id: logs
54+
attributes:
55+
label: Relevant log output
56+
description: Please copy and paste any relevant log output, if any.
57+
render: shell
58+
- type: checkboxes
59+
id: terms
60+
attributes:
61+
label: Code of Conduct
62+
description: By submitting this issue, you agree to follow our [Code of Conduct](CODE_OF_CONDUCT.md)
63+
options:
64+
- label: I agree to follow this project's Code of Conduct
65+
required: true
Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
name: Feature request
2+
description: File a feature request
3+
title: "[FR]: "
4+
labels: ["enhancement"]
5+
body:
6+
- type: markdown
7+
attributes:
8+
value: |
9+
Thanks for taking the time to fill out this feature request!
10+
- type: checkboxes
11+
attributes:
12+
label: Is there an existing issue for this?
13+
description: Please search to see if an issue already exists for this feature request.
14+
options:
15+
label: I have searched the existing issues
16+
required: true
17+
- type: textarea
18+
id: describe-problem
19+
attributes:
20+
label: Describe the problem
21+
description: Is your feature request related to a problem? Please describe.
22+
placeholder: I'm always frustrated when...
23+
validations:
24+
required: true
25+
- type: textarea
26+
id: solution
27+
attributes:
28+
label: Describe the solution
29+
description: Please describe the solution you'd like. A clear and concise description of what you want to happen.
30+
validations:
31+
required: true
32+
- type: textarea
33+
id: context
34+
attributes:
35+
label: Additional context
36+
description: Add any other context or screenshots about the feature request here.
37+
validations:
38+
required: false
39+
- type: checkboxes
40+
id: terms
41+
attributes:
42+
label: Code of Conduct
43+
description: By submitting this issue, you agree to follow our [Code of Conduct](CODE_OF_CONDUCT.md)
44+
options:
45+
- label: I agree to follow this project's Code of Conduct
46+
required: true

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 0 additions & 2 deletions
This file was deleted.
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
name: Pull request
3+
about: Create a pull request
4+
---
5+
Thank you for opening a Pull Request!
6+
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
7+
- [ ] Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
8+
- [ ] Ensure the tests (`melos unit:test-script`)
9+
- [ ] Ensure the analyzer and formatter pass (`melos run format` to automatically apply formatting)
10+
- [ ] Appropriate docs were updated (if necessary)
11+
12+
Fixes #<issue_number_goes_here> 🎯

CONTRIBUTING.md

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,22 @@ _See also: [Contributor Covenant Code of Conduct](https://github.com/FlutterGen/
66

77
* Linux, Mac OS X, or Windows.
88
* Flutter and Dart (Stable channel)
9+
* [Melos](https://melos.invertase.dev/)
910
* Git
1011

12+
## [Melos](https://melos.invertase.dev/getting-started) Installation
13+
Melos can be installed as a global package via pub.dev:
14+
```sh
15+
$ dart pub global activate melos
16+
```
17+
18+
Once installed & setup, Melos needs to be bootstrapped. Bootstrapping has 2 primary roles:
19+
- Installing all package dependencies (internally using pub get).
20+
- Locally linking any packages together.
21+
```sh
22+
$ melos bootstrap
23+
```
24+
1125
## Running the FlutterGen
1226

1327
#### Use as Dart command
@@ -16,24 +30,28 @@ To run `pub get` to make sure its dependencies have been downloaded, and use `da
1630
$ dart packages/command/bin/flutter_gen_command.dart --config example/pubspec.yaml
1731
```
1832

33+
Or melos
34+
```sh
35+
$ melos run example:command
36+
```
37+
1938
#### Use as part of build_runner
2039
```sh
2140
$ cd example
2241
$ flutter packages pub run build_runner build
2342
```
2443

25-
#### Use Intellij
26-
27-
![Run on IDE](./art/run_on_ide.jpg)
28-
44+
Or melos
45+
```sh
46+
$ melos run example:build_runner
47+
```
2948

3049
## Running the tests
3150

3251
To run the unit tests:
3352

3453
```
35-
$ cd packages/core
36-
$ pub run test
54+
$ melos run unit:test
3755
```
3856

3957
## Contributing code

art/run_on_ide.jpg

-13 KB
Binary file not shown.

melos.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ ide:
1212
command:
1313
bootstrap:
1414
# NOTE: https://github.com/invertase/melos/issues/117#issuecomment-1125224468
15-
# A workaround is to run melos clean before melos publish,
15+
# A workaround is to run melos clean before melos publish,
1616
# which removes pubspec_overrides.yamls if they were generated by Melos.
1717
usePubspecOverrides: true
1818

@@ -29,7 +29,7 @@ scripts:
2929
run: |
3030
melos exec -- dart pub run build_runner build --delete-conflicting-outputs
3131
select-package:
32-
ignore:
32+
ignore:
3333
- example
3434
- example_resources
3535
depends-on: 'build_runner'

0 commit comments

Comments
 (0)