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

support for module .mjs files #561

Merged
merged 66 commits into from
Jun 30, 2024
Merged

support for module .mjs files #561

merged 66 commits into from
Jun 30, 2024

Conversation

pelikhan
Copy link
Member

@pelikhan pelikhan commented Jun 21, 2024

  • switch to tsx loader
  • 🔧 The genaiscript.vsix.insiders package file built will no longer include the genaiscript.insiders.vsix file, making the genaiscript.insiders the only build output.
  • 👥 Changes in the third party licenses. Several npm packages have been updated, including @unocss/reset, bson, openai, and multiple @aws-sdk and @smithy packages.
  • 📦 A number of new npm packages have been added to the license list, including gaxios, gcp-metadata, google-auth-library, and googleapis-common.
  • 🗃️ CONTRIBUTING.md has been updated to reflect changes in the built package files.
  • 🆕 Addition of the [email protected] package.
  • ✅ Updating of various dependency versions in docs/package.json for compatibility and functionality improvements.
  • 👀 Minor changes to improve accessibility and correctness in the astro code snippets in the image-alt-text.mdx file.
  • 🔄 Code updates in the automating-scripts.mdx file. Ideal for enhancing the calling function for script automation.
  • 👍 A removal of information about using GenAIScript in IDEs other than VS Code as well as instructions for VSCode Copilot Insiders from the FAQ section.
  • 💡 Adjustment to some case studies for better readability and understanding.
  • 🔄 Updating of the Astro and TypeScript versions in the docs/package.json dependencies.
  • 📝 Various other minor updates and improvements throughout the codebase.

generated by pr-describe

import { Code } from '@astrojs/starlight/components';

import { Image } from "astro:assets"
import { Code } from "@astrojs/starlight/components"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect import statement syntax; semicolons are missing.

generated by pr-docs-review-commit import_statement

import scriptSrc from "../../../../genaisrc/image-alt-text.genai.js?raw"
import src from '../../../assets/debugger.png';
import src from "../../../assets/debugger.png"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect import statement syntax; semicolon is missing.

generated by pr-docs-review-commit import_statement

import { Image } from 'astro:assets';
import src from '../../../assets/debugger.png';
import { Image } from "astro:assets"
import src from "../../../assets/debugger.png"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect import statement syntax; semicolons are missing.

generated by pr-docs-review-commit import_statement

@@ -21,7 +21,7 @@ Use the `> GenAIScript: Create new script...` command in the command palette to
This is an easy script. Assuming the script will take the file as an argument,
you can refer to that argument in `env.files` and tell the LLM what to do with it:

```js title="gen-tweet.genai.js"
```js title="gen-tweet.genai.mjs"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect script file extension; should be .genai.mjs instead of .genai.js.

generated by pr-docs-review-commit incorrect_file_extension

@pelikhan pelikhan merged commit 10982d9 into main Jun 30, 2024
9 checks passed
@pelikhan pelikhan deleted the tsx2 branch June 30, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant