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.

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

bombastictranz/TypScript #5

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Conversation

bombastictranz
Copy link
Owner

Fixes #

Stock control supervisors MSN Money.html
Stock control supervisors MSN Money.html
Stock control supervisors MSN Money,html
Stock control supervisors MSN Money.html
@bombastictranz bombastictranz added bug Something isn't working dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested wontfix This will not be worked on labels Mar 11, 2024
@bombastictranz bombastictranz self-assigned this Mar 11, 2024
Copy link

codeautopilot bot commented Mar 11, 2024

PR summary

The Pull Request (PR) appears to be adding HTML files related to financial data and stock market information from MSN Money. The files include stock quotes, business news, data from stock markets, a portfolio manager and stock watchlist, updates on the S&P/TSX Composite Index, and cryptocurrency insights and price updates. The PR lacks context as there is no associated issue in the backlog, and it's unclear how these additions integrate with the TypeScript project or its documentation.

Suggestion

Given the nature of the TypeScript project, this PR seems out of scope unless it's part of a larger effort to provide financial data handling within TypeScript, which is not mentioned. The PR should be linked to an issue that explains the need for these HTML files and how they fit into the TypeScript project. If these files are not relevant to TypeScript, the PR should be closed. Additionally, the PR title "bombastictranz/TypScript" seems to contain a typo and should be corrected to "TypeScript" if it's indeed related to the TypeScript project.

Disclaimer: This comment was entirely generated using AI. Be aware that the information provided may be incorrect.

Current plan usage: 9.24%

Have feedback or need help?
Discord
Documentation
[email protected]

@bombastictranz bombastictranz merged commit 30f124e into bombastictranz:main Mar 11, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant