Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply prettier to target application. #6

Merged
merged 2 commits into from
Apr 9, 2024
Merged

Apply prettier to target application. #6

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8865744
Update generator.js
mshima Apr 8, 2024
e268271
update tests
mshima Apr 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions generators/migrate/generator.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -412,6 +412,7 @@ export default class extends BaseGenerator {
type: 'target',
cliOptions: (targetCliOptions ? targetCliOptions.split(' ') : undefined) ?? [],
});
await this.applyPrettier({ name: `applying prettier to ${targetApplicationBranch} application`, type: 'target' });
},
});
}
Expand Down
3 changes: 2 additions & 1 deletion generators/migrate/generator.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ describe('SubGenerator migrate of migrate JHipster blueprint', () => {
.replace(new RegExp(escapeRegExp(version), 'g'), 'VERSION'),
).toMatchInlineSnapshot(`
"Merging jhipster_migrate_target into application
apply updated prettier to target application
Migration application generated with JHipster bundled (target)
Initial merge of jhipster_migrate_source branch into application
apply actual application to migration branch
Expand All @@ -74,7 +75,7 @@ describe('SubGenerator migrate of migrate JHipster blueprint', () => {
`);
});
it('generates expected number of commits', async () => {
expect((await createGit().log()).total).toBe(8);
expect((await createGit().log()).total).toBe(9);
});
it('should remove file from updated config', async () => {
assert.noFile('.lintstagedrc.js');
Expand Down
Loading