Skip to content

feat: updates CLI to use https flag - #814

Merged
matt-aitken merged 5 commits into
triggerdotdev:mainfrom
Chigala:feat/support-CLI-https-flag
Jan 12, 2024
Merged

matt-aitken merged 5 commits into
triggerdotdev:mainfrom
Chigala:feat/support-CLI-https-flag

Conversation

@Chigala

@Chigala Chigala commented Dec 27, 2023

Copy link
Copy Markdown
Contributor

Closes #801

✅ Checklist

  • I have followed every step in the contributing guide
  • The PR title follows the convention.
  • I ran and tested the code works

Testing

I started my NextJs server using the --experimental https flag with the nextjs project in the references folder and the used the -https flag on the CLI and it workded, then I also tested the CLI without the flag on the nextJs server not running https and it also worked.

Changelog

NIL

Screenshots

Screenshot 2023-12-26 at 8 39 40 PM

💯
Screenshot 2023-12-26 at 8 40 51 PM

I'll love your feedbacks on this

@changeset-bot

changeset-bot Bot commented Dec 27, 2023

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 3fdcd4c

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Frumba

Frumba commented Jan 12, 2024

Copy link
Copy Markdown

Hey ! Can't wait to get this merged onto main branch to finally use it 🙏

@matt-aitken matt-aitken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The code looks good 👍

This needs testing on:

  • A regular Next.js project
  • A Next.js project with HTTPS enabled

@matt-aitken

Copy link
Copy Markdown
Member

Thanks a lot @Chigala, this is great 😄

@matt-aitken
matt-aitken merged commit bc61d83 into triggerdotdev:main Jan 12, 2024
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.

[TRI-1786] CLI dev command should have a flag for https

3 participants