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

Small fix to TimeSkill minutes and seconds #184

Merged
merged 1 commit into from
Mar 28, 2023
Merged

Small fix to TimeSkill minutes and seconds #184

merged 1 commit into from
Mar 28, 2023

Conversation

RogerBarreto
Copy link
Member

Bugfix

Minutes and Seconds returning wrong information

Contribution Checklist

@RogerBarreto RogerBarreto added bug Something isn't working PR: ready to merge PR has been approved by all reviewers, and is ready to merge. labels Mar 28, 2023
@lemillermicrosoft lemillermicrosoft merged commit ed447a6 into microsoft:main Mar 28, 2023
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Bugfix

Minutes and Seconds returning wrong information
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
### Motivation and Context

Adding docker support to deploy into kubernetes or run in local docker
environment

### Description

Adding docker support to deploy into kubernetes or run in local docker
environment.
A new folder "docker" created under root, to build images, from root
directory, run
`docker build -f docker/webapi/Dockerfile -t repo/chat-copilot-webapi`
`docker build -f docker/webapi/Dockerfile -t repo/chat-copilot-webapp`
A docker-compose file is also created to start whole environment, under
docker directory, just run
`docker-compose up`, it uses .env file to read environment variables, so
make sure webapi and webapp directory has .env file

the manifests to deply chat-copilot into kubernetes environment can be
found here

https://github.com/huangyingting/flux2-gitops/tree/main/apps/base/chat-copilot

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants