Skip to content

Conversation

yann300
Copy link
Contributor

@yann300 yann300 commented Jun 17, 2025

partially_fix #6041

  • always save artifact in the root folder.
  • fix remappings when compiling a contract from the .debug folder.
  • pass the endpoint URL to the etherscan resolver.

@yann300 yann300 requested a review from Aniket-Engg June 17, 2025 07:00
Copy link

netlify bot commented Jun 17, 2025

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 509ce93
🔍 Latest deploy log https://app.netlify.com/projects/remixproject/deploys/685299ce236d37000895f268
😎 Deploy Preview https://deploy-preview-6093--remixproject.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@yann300 yann300 force-pushed the fix_loading_contracts_from_url branch 8 times, most recently from 4d34bc3 to ee7e683 Compare June 17, 2025 21:08
@yann300 yann300 requested a review from bunsenstraat June 18, 2025 08:32
}

if (data.result[0].FileName === '') {
if (data.result[0].FileName === '.sol' && data.result[0].ContractName) {
Copy link
Collaborator

@ioedeveloper ioedeveloper Jun 18, 2025

Choose a reason for hiding this comment

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

Is this an error data.result[0].FileName === '.sol'? or do you meant data.result[0].FileName.endsWith('.sol')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is intended, this API isn't really consistent and in my test case the filename returned by the API ia .sol

@yann300 yann300 requested a review from ioedeveloper June 18, 2025 10:48
@yann300 yann300 force-pushed the fix_loading_contracts_from_url branch from ee7e683 to 509ce93 Compare June 18, 2025 10:49
@yann300 yann300 merged commit c01d28f into master Jun 18, 2025
33 checks passed
@yann300 yann300 deleted the fix_loading_contracts_from_url branch June 18, 2025 11:15
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.

2 participants