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

phone-number.wat entry template is empty #138

Closed
atk opened this issue Jan 2, 2025 · 1 comment · Fixed by #142
Closed

phone-number.wat entry template is empty #138

atk opened this issue Jan 2, 2025 · 1 comment · Fixed by #142

Comments

@atk
Copy link

atk commented Jan 2, 2025

We should at least provide a function signature, e.g.

(module
  (memory (export "mem") 1)

  ;;
  ;; Returns a cleaned valid phone number or an empty string if invalid
  ;;
  ;; @param {i32} offset - The offset of the phone number in linear memory.
  ;; @param {i32} length - The length of the phone number in linear memory.
  ;;
  ;; @returns {(i32,i32)} - Offset and length of cleaned phone number or empty string
  ;;                        in linear memory.
  ;;
  (func (export "clean") (param $offset i32) (param $length i32) (result i32 i32)

  )
)
Copy link

github-actions bot commented Jan 2, 2025

Hello. Thanks for opening an issue on Exercism 🙂

At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.

This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!

If you're interested in learning more about this auto-responder, please read this blog post.

@github-actions github-actions bot closed this as completed Jan 2, 2025
@keiravillekode keiravillekode reopened this Jan 2, 2025
keiravillekode added a commit to keiravillekode/wasm that referenced this issue Jan 5, 2025
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 a pull request may close this issue.

2 participants