hackforla / website

Hack for LA's website
https://www.hackforla.org
GNU General Public License v2.0
286 stars 700 forks source link

ER: Investigate use of TypeScript #6805

Open roslynwythe opened 2 weeks ago

roslynwythe commented 2 weeks ago

Dependencies

Emergent Requirement - Problem

We should explore the possibility of converting our JavaScript code to TypeScript

Issue you discovered this emergent requirement in

Date discovered

Did you have to do something temporarily

Who was involved

@roslynwythe

What happens if this is not addressed

We won't benefit from the code quality and development improvements offered by TypeScript and won't gain experience with the language, which continues to gain adoption.

Resources

typescriptlang.org TypeScript adoption trend

Recommended Action Items

Potential solutions [draft]

Items to consider in the analysis:

github-actions[bot] commented 2 weeks ago

Hi @roslynwythe.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing:

NOTE: Please ignore this comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards. To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

tony1ee commented 2 weeks ago

If we do decide to change to TypeScript, it's a good time to also Adopting a coding style standard/guideline #6807

ExperimentsInHonesty commented 2 weeks ago

@roslynwythe @tony1ee I have added this to our next dev/pm agenda.

ExperimentsInHonesty commented 1 week ago

Moving to icebox and adding dependencies