-
-
Notifications
You must be signed in to change notification settings - Fork 678
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.
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
internal(refactor): improving maintainability for V2 release #1400
internal(refactor): improving maintainability for V2 release #1400
Conversation
Would be great to prepare a simple POSIX script for building and publishing the package. |
Bye bye |
Some things:
|
@MichalLytek
Note that now, before executing
|
…t has been updated to the official index.d.ts types
You can now use import { Hello } from '../../src/World'; NOW: import { Hello } from '~/World';
declare namespace NodeJS {
interface Global {
TypeGraphQLMetadataStorage: import("~/metadata/metadata-storage").MetadataStorage;
// ^
}
} Note that |
@MichalLytek I can't go any further or I'll create conflict with |
@carlocorradini Thank you for doing this work! As someone looking to start contributing to Type-GraphQL, this will go a long way towards my QoL as a contributor. Cheers! |
@mmmeff Thanks 🥳 |
Basically, my publishing workflow is designed in a way that prevents accidental publishing. That's why I had That's why I wanted to have a simple script, that will clear the output folder, build the code, copy it to |
f1354b1
to
bca7e56
Compare
@carlocorradini We should configure But I couldn't make it work 😕 |
@davingee It's added automatically by |
I'll check it out |
831fbea
to
b4c9dfd
Compare
Let's merge it and fix all the issues on the fly! 🎉 🥳 🍾 |
This PR starts a series of iterations for improving the maintainability of the project and trying not to create too many conflicts with
new-major-release
branch.Note that this PR is not ready to be merged (as written before) because the code style must be applied to the new code present in
new-major-release
branch and not themaster
since it can create too many conflicts.Should fix #1106 and create a base for a better coding experience!
What has been changed:
settings.json
to includeformatOnSave
anddefaultFormatter
(prettier).extensions.json
which includes a list of recommended extensions to install for improving experience while coding in VS Code (help new contributors with their first contribution).benchmarks
to match ESLint code style.publish-website.sh
toscripts
directory.publish-website.sh
is POSIX-compliant.scripts
include a__commons.sh
file (sourced bypublish-website.sh
) with utility functions (e.g., logger) and more.TSLint
withESLint
. This has been long awaited! The configuration file must be changed to reflect what @MichalLytek desiders (currently is based on Airbnb code style).action-shellcheck
in CI.package.json
for checking and fixing errors.@MichalLytek Let me know what you think and what you want to change/update/add/delete.
PS: Do we really need
Gulp
?