Closed BrunoM90 closed 7 months ago
github-actions[bot]
commented
7 months ago It seems all right 👌
No new problems were found according to the checks applied
💡 Qodana analysis was run in the pull request mode: only the changed files were checked ☁️ View the detailed Qodana report
guibranco
commented
7 months ago Hi @BrunoM90 first of all, thanks for the great pull request. I have one comment about it, you made your changes in the .csv file, but this file is generated automatically by the MergeTool.
The right place to make these changes is in the json file that the MergeTool uses as the source of truth.
Let me know if you are keen to contribute to that or I can merge this pull request and later make the changes in the proper file.
BrunoM90
commented
7 months ago Hi @guibranco. Ok, I can contribute to that. I will send a new pull request.
guibranco
commented
7 months ago @BrunoM90 thanks!
guibranco
commented
7 months ago I'll close this one, since #344 was already approved.
The websites of the following banks were added to the file: -BNDES -CREDICOAMO -C.SUISSE HEDGING-GRIFFO CV S/A -STATE STREET BR S.A. BCO COMERCIAL -UBS BRASIL CCTVM S.A. -CCM DESP TR NS SC E RS -CONFIDENCE CC S.A. -HAITONG BI DO BRASIL S.A. -B&T CC LTDA. -SISPRIME DO BRASIL - COOP -COOPCENTRAL AILOS -CREDISAN CC -CCCM UNICRED CENTRAL RS -BRK S.A. CFI -POLOCRED SCMEPP LTDA. -CREDISIS - CENTRAL DE COOPERATIVAS DE CRÉDITO LTDA. -CREDIALIANÇA CCR -UNIPRIME COOPCENTRAL LTDA. -PLANNER CV S.A. -RENASCENCA DTVM LTDA -XP INVESTIMENTOS CCTVM S/A -LECCA CFI S.A. -PORTOCRED S.A. - CFI -OLIVEIRA TRUST DTVM S.A. -NEON CTVM S.A. -CENTRAL COOPERATIVA DE CRÉDITO NO ESTADO DO ESPÍRITO SANTO -ADVANCED CC LTDA -BCO RNX S.A.
Issue #342