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

Jump to bottom

Remove unused code and configuration #397

Open
thewilkybarkid opened this issue Aug 11, 2021 · 0 comments
Open

Remove unused code and configuration #397

thewilkybarkid opened this issue Aug 11, 2021 · 0 comments

Comments

@thewilkybarkid
Copy link
Member

There's an awful lot of code and configuration left over from previous versions of the site, and there are either never run or completely unused.
thewilkybarkid added a commit that referenced this issue Aug 11, 2021
@thewilkybarkid
refactor: remove unused database import code
a503118 
The migration from the old site was run once, then turned off. This code has not been needed since and needs to be cleaned up.

Refs #391, #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove unused components
5b1c113 
There's a whole directory full of 'archived' components. Since they are unused, they should be removed. (Since the code is under version control, that's a more suitable archive.)

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove unused imports
0dac056 
This removes import statements that are either unused or commented out.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove unused backend modules
aa6aafa 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove unused schemas
a1d664a 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: moved favicon to the frontend and remove unused assets
b40ce7f 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove unused 'common' code
6f97804 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: simplify code that isn't common
1d2811e 
Moves code from being classed as 'common' to where it's used.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove superflouous facade
de86db5 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 16, 2021
@thewilkybarkid
refactor: remove reference to the frontend from the backend
3c403cd 
Assuming this is a mistake; it's the only place where the backend code references the frontend and vice versa.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 17, 2021
@thewilkybarkid
refactor: remove unused components
247644d 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 17, 2021
@thewilkybarkid
refactor: use plain CSS and resolve dependency tree
3578f25 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 17, 2021
@thewilkybarkid
refactor: remove Sass
081d301 
Following 3578f25 Sass is unused in the project.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 17, 2021
@thewilkybarkid
refactor: remove unused frontend modules
9a4c1be 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 18, 2021
@thewilkybarkid
refactor: remove unused dependencies
bb7ccbd 
Remove dependencies that aren't imported. In some cases, the dependency remains, it's just used by another dependency rather than that the app directly. In others, they're not used at all.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 18, 2021
@thewilkybarkid
refactor: remove unused exports
d307ee3 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 18, 2021
@thewilkybarkid
refactor: remove unused assets
4726255 
This change also moves the 2 remaining ones alongside the other assets.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove superfluous Babel configuration
654b122 
These settings are either unused or included by a preset.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove ASCII art on startup
2e972fb 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: inline environment variables
d6bbe23 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove redundant Babel TypeScript configuration
31d8dfc 
TypeScript files are processed using @parcel/transformer-typescript-tsc, not Babel.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove Babel configuration
35ad67a 
It turns out that the Babel configuration is not needed at all.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
factor: remove duplicate Prettier configuration
3543f20 
Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: extract ESLint and Jest configuration
f2195a0 
Having them separate means that making a change doesn't bust the Docker cache.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove package info
24e30d1 
The package won't be published, so there's no need to include details there.

This change also removes the shebang from the backend entry point, which is redundant since the executable bit is not set.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 19, 2021
@thewilkybarkid
refactor: remove superfluous entries
37a1d25 
I'm not sure what these entries are for; they seem to duplicate the targets, but I can't determine what would use them.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 20, 2021
@thewilkybarkid
refactor: remove unused builds
97852e4 
Only 2 of the scripts need to be built; this change removes those that don't.

Refs #397
thewilkybarkid added a commit that referenced this issue Aug 23, 2021
@thewilkybarkid
refactor: remove unused JSDoc
636412b 
Refs #397
thewilkybarkid added a commit that referenced this issue Sep 7, 2021
@thewilkybarkid
refactor: remove unused function
a637a6d 
It's unused and appears to do nothing anyhow.

Refs #397, #398
thewilkybarkid added a commit that referenced this issue Sep 8, 2021
@thewilkybarkid
refactor: remove unused Cloudflare configuration
e0b6f9b 
Refs #397
thewilkybarkid added a commit that referenced this issue Sep 10, 2021
@thewilkybarkid
fix: restore Axios's dependencies
ddb451d 
I'm not sure why, but e0b6f9b broke npm. Executing an 'npm install' saw it restore Axios's dependencies.

Refs #397
thewilkybarkid added a commit that referenced this issue Sep 10, 2021
@thewilkybarkid
refactor: remove cruft from the entry point
ba1da43 
The app would have been originally generated with create-react-app or similar, which left over various things that are not needed. This change tidies up the entry point.

Refs #397
thewilkybarkid added a commit that referenced this issue Sep 17, 2021
@thewilkybarkid
refactor: remove unused dependency
ea65c71 
Refs #397
thewilkybarkid added a commit that referenced this issue Sep 17, 2021
@thewilkybarkid
refactor: remove superfluous polyfill
b0f0e4a 
As IE is not supported, it's redundant to include a polyfill for it.

Refs #397
thewilkybarkid added a commit that referenced this issue Nov 26, 2021
@thewilkybarkid
refactor: remove unused dependency
d546124 
Refs #397, #399
thewilkybarkid added a commit that referenced this issue Dec 16, 2021
@thewilkybarkid
refactor: remove unused variables
793d978 
Refs #397, #430
thewilkybarkid added a commit that referenced this issue Dec 16, 2021
@thewilkybarkid
refactor: simplify code that isn't common
1b85584 
Moves code from being classed as 'common' to the only place that uses it.

Refs #397, #430
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thewilkybarkid