Skip to content

Conversation

barshimi
Copy link
Contributor

@barshimi barshimi commented Oct 11, 2022

Close: #3396

PR Details

PR Checklist

  • Tests for the changes have been added
  • npm test doesn't throw any error

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

ahlavorato and others added 3 commits October 4, 2022 21:37
…ion-nest-logger-module-amplification-git-service

use amplication nest logger module in amplication-git-service
@barshimi
Copy link
Contributor Author

community PR.
@arielweinberger this PR fail to load @amplication/nest-logger-module inside @amplication-git-service, because it is a library and not package (it doesn't have dockerfile).
Please check this issue again to insure it's valid and possible.

@GreenMachine01
Copy link
Contributor

@overbit @arielweinberger please review.

# Conflicts:
#	libs/data-service-generator/package-lock.json
#	packages/amplication-git-service/package.json
#	packages/amplication-git-service/src/git.module.ts
#	packages/amplication-git-service/src/utils/constants.ts
@nx-cloud
Copy link

nx-cloud bot commented Jan 18, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 020195c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 5 targets

Sent with 💌 from NxCloud.

@yuval-hazaz
Copy link
Member

I am closing this PR due to inactivity
Please reopen if it still needed

@yuval-hazaz yuval-hazaz deleted the merge/4055 branch January 19, 2023 07:59
@overbit overbit restored the merge/4055 branch January 19, 2023 08:37
@overbit
Copy link
Contributor

overbit commented Jan 19, 2023

@yuval-hazaz I actually fix the conflicts yesterday, I'll reopen it.

@overbit overbit reopened this Jan 19, 2023
@yuval-hazaz yuval-hazaz merged commit 9d93f51 into next Jan 22, 2023
@yuval-hazaz yuval-hazaz deleted the merge/4055 branch January 22, 2023 08:00
@EugeneTseitlin
Copy link
Contributor

@overbit please, don't forget to link PR to the issue

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.

Refactor amplication-git-service to use @amplication/nest-logger-module
7 participants